Merge pull request #92 from Dude29/quit-with-focus

Quick fix for ALT+F4 global shortcut
krydos · Jun 29, 2020 · 3b412d4 · 3b412d4
2 parents 0ea2f50 + 7f4a3b5
commit 3b412d4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/main.js b/src/main.js
@@ -137,8 +137,8 @@ function createWindow () {
   });
 
   win.webContents.on('new-window', handleRedirect)
-  // manage size/positio of the window
-  // so it can be restore next time
+  // manage size/position of the window
+  // so it can be restored next time
   mainWindowState.manage(win);
 }
 

diff --git a/src/shortcuts.js b/src/shortcuts.js
@@ -59,6 +59,10 @@ class shortcuts {
 
   registerQuitShortcut() {
     globalShortcut.register(this.shortcutConfig.config['quit'], () => {
+      if(!this.win.isFocused()) {
+        return;
+      }
+
       // isQuitting is important for
       // on('close') event where this variable is checked.
       // In case it is not true then the app just minimized.