I’ve posted some initial designs for the keyboard shortcut editor panel. I’ll start implementing the UI from feedback given for the designs.
I’ve also added an utility method to convert cross-platform Electron Accelerators to match the current platform. For example, the shortcut CommandOrControl+N
will be shown as Command+N
or Cmd+N
for macOS users, and Control+N
or Ctrl+N
for everyone else.
Unit tests have also been improved for testing following cases:
- Ensure no warnings are logged on a successful loading of keymap-desktop.json file.
- Keymap configurations in the JSON file containing invalid Accelerators should be rejected (i.e.
{ "newNoteItem": "options", "accelerator": "F16" }
) - Invalid commands should also be rejected (i.e.
{ "command": "totallyInvalidCommand", "accelerator": "Control+Shift+M" }
) - When the JSON file contains a duplicate, the entire file is rejected and all shortcuts should be their default values.