From 38c02509eadd6e9ff8fa62c48e0e9450d375068d Mon Sep 17 00:00:00 2001
From: Tom MacWright <tom@macwright.org>
Date: Tue, 22 Jan 2013 10:39:10 -0500
Subject: [PATCH] Do not tolerate modifiers for action shortcuts.

Tolerates id-equality for selecting features.
---
 js/id/id.js           | 3 +++
 js/id/modes/select.js | 4 +++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/js/id/id.js b/js/id/id.js
index 3d9004bfb..31ff26836 100644
--- a/js/id/id.js
+++ b/js/id/id.js
@@ -186,15 +186,18 @@ window.iD = function(container) {
 
         map.keybinding()
             .on('a', function(evt, mods) {
+                if (mods) return;
                 controller.enter(iD.modes.AddArea());
             })
             .on('⌫.prevent_navigation', function(evt, mods) {
                 evt.preventDefault();
             })
             .on('p', function(evt, mods) {
+                if (mods) return;
                 controller.enter(iD.modes.AddPoint());
             })
             .on('l', function(evt, mods) {
+                if (mods) return;
                 controller.enter(iD.modes.AddLine());
             })
             .on('z', function(evt, mods) {
diff --git a/js/id/modes/select.js b/js/id/modes/select.js
index 3b9667de3..9397b1e98 100644
--- a/js/id/modes/select.js
+++ b/js/id/modes/select.js
@@ -138,7 +138,9 @@ iD.modes.Select = function(entity) {
         });
 
         surface.selectAll("*")
-            .filter(function (d) { return d === entity; })
+            .filter(function (d) {
+                return d && entity && d.id === entity.id;
+            })
             .classed('selected', true);
     };