Fix a bunch of key handling related bugs in one swoop.

We were incorrectly generating valid first keys and completion keys for <c-x> type commands. This closes issue #85 and the alt+d override issue. 

1 files changed, 11 insertions, 8 deletions

diff --git a/vimiumFrontend.js b/vimiumFrontend.js
index bac5d8ae..f67768f7 100644
--- a/vimiumFrontend.js
+++ b/vimiumFrontend.js
@@ -287,14 +287,17 @@ function onKeydown(event) {
       return;
     }
 
-    if (event.shiftKey)
-      keyChar = keyChar.toUpperCase();
-    if (event.ctrlKey)
-      keyChar = "<c-" + keyChar + ">";
-    if (event.metaKey)
-      keyChar = "<m-" + keyChar + ">";
-    if (event.altKey)
-      keyChar = "<a-" + keyChar + ">";
+    if (keyChar != "") // Again, ignore just modifiers. Maybe this should replace the keyCode > 31 condition.
+    {
+      if (event.shiftKey)
+        keyChar = keyChar.toUpperCase();
+      else if (event.ctrlKey)
+        keyChar = "<c-" + keyChar + ">";
+      else if (event.metaKey)
+        keyChar = "<m-" + keyChar + ">";
+      else if (event.altKey)
+        keyChar = "<a-" + keyChar + ">";
+    }
   }
 
   if (insertMode && isEscape(event))