From d441cd92c3bc23e67988bc9f01b831eb60cc31d1 Mon Sep 17 00:00:00 2001
From: Bryan Housel <bryan@mapbox.com>
Date: Thu, 20 Apr 2017 22:16:11 -0400
Subject: [PATCH] Smaller nudge regions in drag_node and move (closes #3956)

---
 modules/modes/drag_node.js | 33 ++++++++++++++++++++-------------
 modules/modes/move.js      | 30 +++++++++++++++++++-----------
 2 files changed, 39 insertions(+), 24 deletions(-)

diff --git a/modules/modes/drag_node.js b/modules/modes/drag_node.js
index 782427218..4643b1005 100644
--- a/modules/modes/drag_node.js
+++ b/modules/modes/drag_node.js
@@ -45,17 +45,17 @@ export function modeDragNode(context) {
     }
 
     function edge(point, size) {
-        var pad = [30, 100, 30, 100],
+        var pad = [80, 20, 50, 20],   // top, right, bottom, left
             x = 0,
             y = 0;
 
-        if (point[0] > size[0] - pad[0])
+        if (point[0] > size[0] - pad[1])
             x = -10;
-        if (point[0] < pad[2])
+        if (point[0] < pad[3])
             x = 10;
-        if (point[1] > size[1] - pad[1])
+        if (point[1] > size[1] - pad[2])
             y = -10;
-        if (point[1] < pad[3])
+        if (point[1] < pad[0])
             y = 10;
 
         if (x || y) {
@@ -76,8 +76,10 @@ export function modeDragNode(context) {
 
 
     function stopNudge() {
-        if (nudgeInterval) window.clearInterval(nudgeInterval);
-        nudgeInterval = null;
+        if (nudgeInterval) {
+            window.clearInterval(nudgeInterval);
+            nudgeInterval = null;
+        }
     }
 
 
@@ -159,10 +161,12 @@ export function modeDragNode(context) {
             loc = context.projection.invert(currMouse),
             d = datum();
 
-        if (d.type === 'node' && d.id !== entity.id) {
-            loc = d.loc;
-        } else if (d.type === 'way' && !d3.select(d3.event.sourceEvent.target).classed('fill')) {
-            loc = geoChooseEdge(context.childNodes(d), context.mouse(), context.projection).loc;
+        if (!nudgeInterval) {
+            if (d.type === 'node' && d.id !== entity.id) {
+                loc = d.loc;
+            } else if (d.type === 'way' && !d3.select(d3.event.sourceEvent.target).classed('fill')) {
+                loc = geoChooseEdge(context.childNodes(d), context.mouse(), context.projection).loc;
+            }
         }
 
         context.replace(
@@ -181,8 +185,11 @@ export function modeDragNode(context) {
 
         doMove(entity);
         var nudge = edge(d3.event.point, context.map().dimensions());
-        if (nudge) startNudge(entity, nudge);
-        else stopNudge();
+        if (nudge) {
+            startNudge(entity, nudge);
+        } else {
+            stopNudge();
+        }
     }
 
 
diff --git a/modules/modes/move.js b/modules/modes/move.js
index 114063521..d8d392834 100644
--- a/modules/modes/move.js
+++ b/modules/modes/move.js
@@ -51,21 +51,24 @@ export function modeMove(context, entityIDs, baseGraph) {
 
 
     function edge(point, size) {
-        var pad = [30, 100, 30, 100],
+        var pad = [80, 20, 50, 20],   // top, right, bottom, left
             x = 0,
             y = 0;
 
-        if (point[0] > size[0] - pad[0])
+        if (point[0] > size[0] - pad[1])
             x = -10;
-        if (point[0] < pad[2])
+        if (point[0] < pad[3])
             x = 10;
-        if (point[1] > size[1] - pad[1])
+        if (point[1] > size[1] - pad[2])
             y = -10;
-        if (point[1] < pad[3])
+        if (point[1] < pad[0])
             y = 10;
 
-        if (x || y) return [x, y];
-        else return null;
+        if (x || y) {
+            return [x, y];
+        } else {
+            return null;
+        }
     }
 
 
@@ -100,16 +103,21 @@ export function modeMove(context, entityIDs, baseGraph) {
 
 
     function stopNudge() {
-        if (nudgeInterval) window.clearInterval(nudgeInterval);
-        nudgeInterval = null;
+        if (nudgeInterval) {
+            window.clearInterval(nudgeInterval);
+            nudgeInterval = null;
+        }
     }
 
 
     function move() {
         doMove();
         var nudge = edge(context.mouse(), context.map().dimensions());
-        if (nudge) startNudge(nudge);
-        else stopNudge();
+        if (nudge) {
+            startNudge(nudge);
+        } else {
+            stopNudge();
+        }
     }