refactor(js): simplify a bit keyboard_handler.js

- Mark two methods as static
- Use a `switch-case` construct instead of an Object and a loop.

internal/ui/static/js/keyboard_handler.js

@@ -12,8 +12,8 @@ class KeyboardHandler {
 
     listen() {
         document.onkeydown = (event) => {
-            const key = this.getKey(event);
+            const key = KeyboardHandler.getKey(event);
-            if (this.isEventIgnored(event, key) || this.isModifierKeyDown(event)) {
+            if (this.isEventIgnored(event, key) || KeyboardHandler.isModifierKeyDown(event)) {
                 return;
             }
 
@@ -51,25 +51,18 @@ class KeyboardHandler {
             (this.queue.length < 1 && !this.triggers.has(key));
     }
 
-    isModifierKeyDown(event) {
+    static isModifierKeyDown(event) {
         return event.getModifierState("Control") || event.getModifierState("Alt") || event.getModifierState("Meta");
     }
 
-    getKey(event) {
+    static getKey(event) {
-        const mapping = {
+        switch (event.key) {
-            'Esc': 'Escape',
+        case 'Esc': return 'Escape';
-            'Up': 'ArrowUp',
+        case 'Up': return 'ArrowUp';
-            'Down': 'ArrowDown',
+        case 'Down': return 'ArrowDown';
-            'Left': 'ArrowLeft',
+        case 'Left': return 'ArrowLeft';
-            'Right': 'ArrowRight'
+        case 'Right': return 'ArrowRight';
-        };
+        default: return event.key;
-
-        for (const key in mapping) {
-            if (mapping.hasOwnProperty(key) && key === event.key) {
-                return mapping[key];
-            }
         }
-
-        return event.key;
     }
 }