From fd5874a96b1bbee56ba5fd0bacb719a861a157d3 Mon Sep 17 00:00:00 2001
From: John Firebaugh <john.firebaugh@gmail.com>
Date: Thu, 6 Dec 2012 10:38:51 -0500
Subject: [PATCH 1/2] Remove unnecessary redraws

---
 js/id/renderer/map.js | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/js/id/renderer/map.js b/js/id/renderer/map.js
index b3e2a54c1..38d4a8d78 100644
--- a/js/id/renderer/map.js
+++ b/js/id/renderer/map.js
@@ -38,13 +38,10 @@ iD.Map = function() {
 
                 var to = projection.invert([d3.event.x, d3.event.y]);
                 history.replace(iD.actions.Move(entity, to));
-
-                redraw();
             })
             .on('dragend', function () {
                 if (!dragEnabled || !dragging) return;
                 dragging = undefined;
-                redraw();
             }),
         background = iD.Background()
             .projection(projection)

From 1c5878fa49ad326a7841db479af3d1989c56f810 Mon Sep 17 00:00:00 2001
From: John Firebaugh <john.firebaugh@gmail.com>
Date: Thu, 6 Dec 2012 10:49:22 -0500
Subject: [PATCH 2/2] Highlight roads when hovering stroke (fixes #165)

---
 css/map.css           |  2 +-
 js/id/renderer/map.js | 24 +++++++++++++++++-------
 2 files changed, 18 insertions(+), 8 deletions(-)

diff --git a/css/map.css b/css/map.css
index 06eb66c2e..c2d7f6693 100644
--- a/css/map.css
+++ b/css/map.css
@@ -48,7 +48,7 @@ path.casing {
     pointer-events:none;
 }
 
-path.casing:hover {
+path.casing.hover {
     stroke:#FF0F0F !important;
     opacity:0.8;
 }
diff --git a/js/id/renderer/map.js b/js/id/renderer/map.js
index 38d4a8d78..b79fc1346 100644
--- a/js/id/renderer/map.js
+++ b/js/id/renderer/map.js
@@ -2,7 +2,7 @@ iD.Map = function() {
     var connection, history,
         dimensions = [],
         dispatch = d3.dispatch('move'),
-        selection = null,
+        selection = null, hover = null,
         translateStart,
         keybinding,
         projection = d3.geo.mercator(),
@@ -84,8 +84,8 @@ iD.Map = function() {
             .attr('clip-path', 'url(#clip)');
 
         r = surface.append('g')
-            .on('mouseover', nameHoverIn)
-            .on('mouseout', nameHoverOut)
+            .on('mouseover', hoverIn)
+            .on('mouseout', hoverOut)
             .attr('clip-path', 'url(#clip)');
 
         g = ['fill', 'casing', 'stroke', 'text', 'hit', 'temp'].reduce(function(mem, i) {
@@ -101,6 +101,7 @@ iD.Map = function() {
     }
 
     function pxCenter() { return [dimensions[0] / 2, dimensions[1] / 2]; }
+    function classHover(d) { return d.id === hover; }
     function classActive(d) { return d.id === selection; }
     function getline(d) { return d._line; }
     function key(d) { return d.id; }
@@ -282,11 +283,13 @@ iD.Map = function() {
         casings.exit().remove();
         casings.enter().append('path')
             .attr('class', class_casing)
+            .classed('hover', classHover)
             .classed('active', classActive);
         casings
             .order()
             .attr('d', getline)
             .attr('class', class_casing)
+            .classed('hover', classHover)
             .classed('active', classActive);
     }
 
@@ -295,12 +298,19 @@ iD.Map = function() {
         drawVector(iD.util.trueObj(Object.keys(result.entities)));
     }
 
-    function nameHoverIn() {
-        var entity = d3.select(d3.event.target).data();
-        if (entity) d3.select('.messages').text(entity[0].tags.name || '#' + entity[0].id);
+    function hoverIn() {
+        var entity = d3.select(d3.event.target).datum();
+        hover = entity.id;
+        drawVector(iD.util.trueObj([hover]));
+        d3.select('.messages').text(entity.tags.name || '#' + entity.id);
     }
 
-    function nameHoverOut() { d3.select('.messages').text(''); }
+    function hoverOut() {
+        var oldHover = hover;
+        hover = null;
+        drawVector(iD.util.trueObj([oldHover]));
+        d3.select('.messages').text('');
+    }
 
     function zoomPan() {
         if (d3.event && d3.event.sourceEvent.type === 'dblclick') {