Merge pull request #1881 from Tyriar/1880_key_repeat

Don't print keys that print more than a single char
xtermjs · Jan 7, 2019 · 509ce5f · 509ce5f
2 parents 0b8c320 + 7a416aa
commit 509ce5f
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/core/input/Keyboard.ts b/src/core/input/Keyboard.ts
@@ -349,9 +349,8 @@ export function evaluateKeyboardEvent(
         if (ev.keyCode === 65) { // cmd + a
           result.type = KeyboardResultType.SELECT_ALL;
         }
-      } else if (ev.key && !ev.ctrlKey && !ev.altKey && !ev.metaKey &&
-          ev.keyCode >= 48 && ev.keyCode !== 144 && ev.keyCode !== 145) {
-        // Include only keys that that result in a character; don't include num lock and scroll lock
+      } else if (ev.key && !ev.ctrlKey && !ev.altKey && !ev.metaKey && ev.keyCode >= 48 && ev.key.length === 1) {
+        // Include only keys that that result in a _single_ character; don't include num lock, volume up, etc.
         result.key = ev.key;
       }
       break;