Use code instead of key (#1234)

* Update KeyboardMouseInputField to use .code instead of .key

* Update Display to use .code instead of .key

* Remove unused function

2 files changed, 9 insertions, 14 deletions

diff --git a/ext/mixed/js/display.js b/ext/mixed/js/display.js
index 6a1ebba6..e361a3a1 100644
--- a/ext/mixed/js/display.js
+++ b/ext/mixed/js/display.js
@@ -140,14 +140,14 @@ class Display extends EventDispatcher {
             {key: 'Home',      modifiers: ['alt'],  action: 'firstEntry'},
             {key: 'ArrowUp',   modifiers: ['alt'],  action: 'previousEntry'},
             {key: 'ArrowDown', modifiers: ['alt'],  action: 'nextEntry'},
-            {key: 'B',         modifiers: ['alt'],  action: 'historyBackward'},
-            {key: 'F',         modifiers: ['alt'],  action: 'historyForward'},
-            {key: 'K',         modifiers: ['alt'],  action: 'addNoteKanji'},
-            {key: 'E',         modifiers: ['alt'],  action: 'addNoteTermKanji'},
-            {key: 'R',         modifiers: ['alt'],  action: 'addNoteTermKana'},
-            {key: 'P',         modifiers: ['alt'],  action: 'playAudio'},
-            {key: 'V',         modifiers: ['alt'],  action: 'viewNote'},
-            {key: 'C',         modifiers: ['ctrl'], action: 'copyHostSelection'}
+            {key: 'KeyB',      modifiers: ['alt'],  action: 'historyBackward'},
+            {key: 'KeyF',      modifiers: ['alt'],  action: 'historyForward'},
+            {key: 'KeyK',      modifiers: ['alt'],  action: 'addNoteKanji'},
+            {key: 'KeyE',      modifiers: ['alt'],  action: 'addNoteTermKanji'},
+            {key: 'KeyR',      modifiers: ['alt'],  action: 'addNoteTermKana'},
+            {key: 'KeyP',      modifiers: ['alt'],  action: 'playAudio'},
+            {key: 'KeyV',      modifiers: ['alt'],  action: 'viewNote'},
+            {key: 'KeyC',      modifiers: ['ctrl'], action: 'copyHostSelection'}
         ]);
         this.registerMessageHandlers([
             ['setMode', {async: false, handler: this._onMessageSetMode.bind(this)}]
@@ -273,7 +273,7 @@ class Display extends EventDispatcher {
     }
 
     onKeyDown(e) {
-        const key = DocumentUtil.getKeyFromEvent(e);
+        const key = e.code;
         const handlers = this._hotkeys.get(key);
         if (typeof handlers === 'undefined') { return false; }
 
diff --git a/ext/mixed/js/document-util.js b/ext/mixed/js/document-util.js
index 42d3556b..0c0a2153 100644
--- a/ext/mixed/js/document-util.js
+++ b/ext/mixed/js/document-util.js
@@ -227,11 +227,6 @@ class DocumentUtil {
         return buttons;
     }
 
-    static getKeyFromEvent(event) {
-        const key = event.key;
-        return (typeof key === 'string' ? (key.length === 1 ? key.toUpperCase() : key) : '');
-    }
-
     static addFullscreenChangeEventListener(onFullscreenChanged, eventListenerCollection=null) {
         const target = document;
         const options = false;