Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This pr initially started as adding
Clone Note
to the menu bar and having a keyboard shortcut for it. However once I got started I thought that the menu items needed some restructuring and some of the keyboard shortcuts needed to be more platform friendly (i.e. using Command on macOS vs Control on Windows\Linux). Some of the keyboard shortcut changes are listed below.Changed shortcuts
Focus Note: On macOS its now
Command+E
otherwise it remainsCtrl+E
Clone Note: On macOS its
Command+D
otherwise it isCtrl+D
I made an attempt on bringing back the
Delete
key for non macOS machines but you can't have multiple accelerators on the same menuItem and I was too lazy to code it elsewhere 🤷♂️ I have also removed theCtrl+D
shortcut for deleting a note since the global shortcut is different as of this pr #2452 and I needed it for Clone Note 😄I also noticed from this issue #2245 that the keyboard shortcuts
wiki
is seriously outdated. I can't update the wiki sadly but I hope that someone with power can do so after this PR is merged in to prevent any future confusion :)What the menu items will now look like
Clone Note proof
NOTE: I have not tested on either Linux or Windows can someone please test before merging. 🙏
Issue fixed
Type of changes
Checklist: