Skip to content

Commit

Permalink
fix(Windows): Fix shortcuts for closing, minimizing and quitting the app
Browse files Browse the repository at this point in the history
  • Loading branch information
adlk committed Mar 26, 2018
1 parent a1e0a29 commit f720d30
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions src/lib/Menu.js
Original file line number Diff line number Diff line change
Expand Up @@ -431,14 +431,14 @@ const _titleBarTemplateFactory = intl => [
submenu: [
{
label: intl.formatMessage(menuItems.minimize),
accelerator: 'Alt+M',
accelerator: 'Ctrl+M',
click(menuItem, browserWindow) {
browserWindow.minimize();
},
},
{
label: intl.formatMessage(menuItems.close),
accelerator: 'Alt+W',
accelerator: 'Ctrl+W',
click(menuItem, browserWindow) {
browserWindow.close();
},
Expand Down Expand Up @@ -643,10 +643,7 @@ export default class FranzMenu {
},
{
label: intl.formatMessage(menuItems.quit),
accelerator: 'Alt+F4',
click: () => {

This comment has been minimized.

Copy link
@dabalroman

dabalroman Apr 12, 2018

Contributor

Sience last update "Quit" option under File menu item isn't working.
It's probably because lines 647 - 649 were removed from Menu.js:

click: () => {
app.quit();
},

Change requested at #888

app.quit();
},
role: 'quit',
},
];

Expand Down

0 comments on commit f720d30

Please sign in to comment.