Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Fix quit from hamburger menu
Browse files Browse the repository at this point in the history
Auditors: @aekeus

Fix #3310
  • Loading branch information
bbondy committed Aug 23, 2016
1 parent ebcf37f commit 70338ea
Showing 1 changed file with 10 additions and 6 deletions.
16 changes: 10 additions & 6 deletions app/common/commonMenu.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,13 +60,17 @@ module.exports.sendToFocusedWindow = (focusedWindow, message) => {
}
}

module.exports.quitMenuItem = () => {
return {
label: locale.translation('quitApp'),
accelerator: 'CmdOrCtrl+Q',
click: app.quit
module.exports.quitMenuItem = () => ({
label: locale.translation('quitApp'),
accelerator: 'CmdOrCtrl+Q',
click: function () {
if (process.type === 'browser') {
app.quit()
} else {
electron.ipcRenderer.send(messages.QUIT_APPLICATION)
}
}
}
})

module.exports.newTabMenuItem = (parentFrameKey) => {
return {
Expand Down

1 comment on commit 70338ea

@aekeus
Copy link
Member

@aekeus aekeus commented on 70338ea Aug 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++ confirmed

Please sign in to comment.