-
Notifications
You must be signed in to change notification settings - Fork 414
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add keyboard shortcut to quit the app on Windows #1202
Changes from 5 commits
be559be
5205bb5
f2af163
97d47e0
e70d8f2
6db06a0
b50606b
572d68d
4b52795
0a4f781
97fbef8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -44,7 +44,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/). | |
* LBRY URLs not working on Linux ([#1120](https://github.com/lbryio/lbry-app/issues/1120)) | ||
* Fix Windows notifications not showing ([#1145](https://github.com/lbryio/lbry-app/pull/1145)) | ||
* Fix export issues ([#1163](https://github.com/lbryio/lbry-app/pull/1163)) | ||
|
||
* Fix Keyboard Shortcut to Quit APP ([#1163](https://github.com/lbryio/lbry-app/pull/1202)) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please remove this line. |
||
|
||
|
||
## [0.20.0] - 2018-01-30 | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -42,6 +42,11 @@ export default () => { | |
} | ||
}, | ||
}, | ||
{ | ||
label: 'Close', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove the label "Close" as it should be automatically specified by the role. |
||
accelerator: 'CmdOrCtrl+Q', | ||
role: 'quit', | ||
}, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This shouldn't be under the help menu. Add it to a new submenu "File". |
||
{ | ||
label: 'Frequently Asked Questions', | ||
click: () => { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rename this as "Add keyboard shortcut to quit the app on Windows" under an unreleased section and not under "Fix".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@IGassmann i cannot see where a unreleased section
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have to add one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@IGassmann like this guys
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
572d68d done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It also needs to be under an Added subsection. Also, notice that the pull request number doesn't match the link number.