Fix scenarios where key is undefined

Author: CompuIves

packages/app/src/app/store/providers/KeybindingManager.js

@@ -31,6 +31,10 @@ function handleKeyDown(controller, e) {
 
   const key = normalizeKey(e);
 
+  if (!key) {
+    return;
+  }
+
   // First we check if we have any pending secondary bindings to identify
   if (state.pendingSecondaryBindings.length) {
     // We filter out any hits by verifying that the current key matches the next

packages/common/utils/keybindings.js

@@ -5,15 +5,19 @@ const metaKey = isMac ? 'Meta' : 'Alt';
 const metaOrCtrlKey = isMac ? 'Meta' : 'Control';
 
 export function normalizeKey(e: KeyboardEvent) {
-  if (e.key.split('').length === 1) {
-    const key = String.fromCharCode(e.keyCode).toUpperCase();
-    if (key === ' ') {
-      return 'Space';
+  if (e.key) {
+    if (e.key.split('').length === 1) {
+      const key = String.fromCharCode(e.keyCode).toUpperCase();
+      if (key === ' ') {
+        return 'Space';
+      }
+      return key;
     }
-    return key;
+
+    return e.key;
   }
 
-  return e.key;
+  return undefined;
 }
 
 export function formatKey(key: string) {