-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add delete hotkey #2452
Add delete hotkey #2452
Conversation
Setting the default hotkeys to be more consistent on OS X (using the Alt notation as opposed to the Option notation).
browser/main/lib/ConfigManager.js
Outdated
@@ -24,7 +24,8 @@ export const DEFAULT_CONFIG = { | |||
amaEnabled: true, | |||
hotkey: { | |||
toggleMain: OSX ? 'Command + Alt + L' : 'Super + Alt + E', | |||
toggleMode: OSX ? 'Command + Option + M' : 'Ctrl + M' | |||
toggleMode: OSX ? 'Command + Alt + M' : 'Ctrl + M', | |||
deleteNote: OSX ? 'Command + Alt + Backspace' : 'Ctrl + Alt + Backspace' |
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.
This hotkey is pure evil on Linux. It close all applications and go back to the login screen. Can you change it to Ctrl + Shift + Backspace
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.
Hahahaha yes
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.
LGTM 🎉
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.
@jacobherrington Could you fix the conflicts?
@Rokt33r Fixed, pretty sure that's right :) |
Would love to get some testing on this one 👍
Solves #2446