From 0c35b8f9ab73370de0037d99c520a02518be1782 Mon Sep 17 00:00:00 2001
From: Bryan Housel <bryan@mapbox.com>
Date: Mon, 17 Oct 2016 21:28:37 -0400
Subject: [PATCH] Fix bug in partial label redraw, tighten margins around line
 labels

---
 modules/svg/labels.js | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/modules/svg/labels.js b/modules/svg/labels.js
index ed2ce6c32..d91e5dd2d 100644
--- a/modules/svg/labels.js
+++ b/modules/svg/labels.js
@@ -185,7 +185,7 @@ export function svgLabels(projection, context) {
 
 
     function drawCollisionBoxes(selection, rtree, which) {
-        var showDebug = context.getDebug('collision'),
+        var showDebug = true, //context.getDebug('collision'),
             classes = 'debug ' + which + ' ' +
                 (which === 'debug-skipped' ? 'orange' : 'yellow');
 
@@ -291,6 +291,7 @@ export function svgLabels(projection, context) {
             bboxes = {};
         } else {
             for (i = 0; i < entities.length; i++) {
+                entity = entities[i];
                 bbox = bboxes[entity.id];
                 if (bbox) {
                     rdrawn.remove(bbox);
@@ -411,8 +412,9 @@ export function svgLabels(projection, context) {
             if (length < width + 20) return;
 
             // % along the line to attempt to place the label
-            var lineOffsets = [50, 45, 55, 40, 60, 35, 65, 30, 70, 25,
-                75, 20, 80, 15, 95, 10, 90, 5, 95];
+            var lineOffsets = [50, 45, 55, 40, 60, 35, 65, 30, 70,
+                               25, 75, 20, 80, 15, 95, 10, 90, 5, 95];
+            var margin = 2;
 
             for (var i = 0; i < lineOffsets.length; i++) {
                 var offset = lineOffsets[i],
@@ -424,10 +426,10 @@ export function svgLabels(projection, context) {
                 var sub = subpath(nodes, start, start + width),
                     rev = reverse(sub),
                     bbox = {
-                        minX: Math.min(sub[0][0], sub[sub.length - 1][0]) - 10,
-                        minY: Math.min(sub[0][1], sub[sub.length - 1][1]) - 10,
-                        maxX: Math.max(sub[0][0], sub[sub.length - 1][0]) + 20,
-                        maxY: Math.max(sub[0][1], sub[sub.length - 1][1]) + 30
+                        minX: Math.min(sub[0][0], sub[sub.length - 1][0]) - margin,
+                        minY: Math.min(sub[0][1], sub[sub.length - 1][1]) - margin,
+                        maxX: Math.max(sub[0][0], sub[sub.length - 1][0]) + margin,
+                        maxY: Math.max(sub[0][1], sub[sub.length - 1][1]) + margin
                     };
 
                 if (rev) {