Added confirmation dialog on app exit

src/dialog.js

@@ -36,6 +36,15 @@ class Dialog {
     );
   }
 
+  _exit() {
+    return this._create({
+      buttons: ['Exit', 'Dismiss'],
+      detail: 'Are you sure you want to exit?',
+      message: 'Exit Tusk',
+      title: 'Tusk - Exit Confirmation'
+    });
+  }
+
   _signOut() {
     return this._create({
       buttons: ['Sign Out', 'Dismiss'],
@@ -69,6 +78,12 @@ class Dialog {
     }
   }
 
+  confirmExit() {
+    if (this._exit() === 0) {
+      app.quit();
+    }
+  }
+
   confirmRestart() {
     if (this._restart() === 0) {
       app.quit();

src/menu/app.js

@@ -1,5 +1,6 @@
 'use strict';
 const {app} = require('electron');
+const dialog = require('./../dialog');
 
 module.exports = {
   label: app.getName(),
@@ -18,7 +19,10 @@ module.exports = {
     }, {
       type: 'separator'
     }, {
-      role: 'quit'
+      label: 'Exit',
+      click() {
+        dialog.confirmExit();
+      }
     }
   ]
 };

src/menu/file.js

@@ -180,8 +180,11 @@ module.exports = {
         dialog.confirmSignOut();
       }
     }, {
-      role: 'quit',
-      visible: !is.darwin
+      label: 'Exit',
+      visible: !is.darwin,
+      click() {
+        dialog.confirmExit();
+      }
     }
   ]
 };

src/tray.js

@@ -1,8 +1,9 @@
 'use strict';
 const {app, Menu, shell, Tray} = require('electron');
 const {is} = require('./util');
-const settings = require('./settings');
+const dialog = require('./dialog');
 const file = require('./file');
+const settings = require('./settings');
 const url = require('./url');
 const win = require('./win');
 
@@ -101,7 +102,10 @@ function create() {
     }, {
       type: 'separator'
     }, {
-      role: 'quit'
+      label: 'Exit',
+      click() {
+        dialog.confirmExit();
+      }
     }
   ]);