Merge pull request #7162 from mananjadhav/fix/change-keyboard-shortcu…

…ts-hotkey fix: Changed keyboard shortcuts hotkey
Expensify · Jan 12, 2022 · 3c14145 · 3c14145
2 parents f3b3bfe + d9b4a98
commit 3c14145
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/desktop/main.js b/desktop/main.js
@@ -103,7 +103,7 @@ const updateAppMenuItem = new MenuItem({
 // Defines the system-level menu item for opening keyboard shortcuts modal
 const keyboardShortcutsMenu = new MenuItem({
     label: 'View Keyboard Shortcuts',
-    accelerator: 'CmdOrCtrl+?',
+    accelerator: 'CmdOrCtrl+I',
 });
 
 // Actual auto-update listeners

diff --git a/src/CONST.js b/src/CONST.js
@@ -152,7 +152,7 @@ const CONST = {
         },
         SHORTCUT_MODAL: {
             descriptionKey: 'openShortcutDialog',
-            shortcutKey: '?',
+            shortcutKey: 'I',
             modifiers: ['CTRL'],
         },
         ESCAPE: {

diff --git a/src/libs/KeyboardShortcut/index.js b/src/libs/KeyboardShortcut/index.js
@@ -104,8 +104,6 @@ function getKeyCode(key) {
             return 13;
         case 'Escape':
             return 27;
-        case '?':
-            return 191;
         default:
             return key.charCodeAt(0);
     }