Alt + left/right arrow keys and Alt + printscreen no longer show the …

…menu (windows only) Properly fixes #5775 Fixes #7566 Auditors: @srirambv, @jonathansampson
brave · Mar 14, 2017 · 981249a · 981249a
1 parent ae1d419
commit 981249a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/app/common/constants/keyCodes.js b/app/common/constants/keyCodes.js
@@ -19,7 +19,8 @@ const KeyCodes = {
   F12: 123,
   NUMPAD_PLUS: 107,
   NUMPAD_MINUS: 109,
-  TAB: 9
+  TAB: 9,
+  PRINT_SCREEN: 44
 }
 
 module.exports = KeyCodes
diff --git a/js/components/main.js b/js/components/main.js
@@ -147,6 +147,12 @@ class Main extends ImmutableComponent {
       document.addEventListener('keyup', (e) => {
         const customTitlebar = this.customTitlebar
         switch (e.which) {
+          case keyCodes.LEFT:
+          case keyCodes.RIGHT:
+          case keyCodes.PRINT_SCREEN:
+            // these keys don't register key down when pressed in combination w/ ALT
+            this.lastKeyPressed = e.which
+            break
           case keyCodes.ALT:
             /*
              Only show/hide the menu if: