mirror of
https://github.com/BoostIO/BoostNote-App.git
synced 2026-04-26 21:05:55 +03:00
[PR #899] [MERGED] Add basic keymap handling #984
Labels
No labels
android 🤖
assigned to core 🦹
bug 🐛
documentation 📚
documentation 📚
duplicate 🚫
external issue 🔼
external issue 🔼
feature request 🌟
funded on issuehunt 💵
help wanted 🆘
improvement request 🔨
improvement request 🔨
ios 🍎
mobile 📱
needs investigation 🔬
needs more info ℹ️
needs specs 📐
plugin idea 🔌
plugin idea 🔌
poll 🗳️
pull-request
question ❓
rewarded on issuehunt 🎁
security issue 🔑
won’t fix ❌
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/BoostNote-App#984
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/BoostIO/BoostNote-App/pull/899
Author: @Komediruzecki
Created: 4/6/2021
Status: ✅ Merged
Merged: 4/29/2021
Merged by: @Rokt33r
Base:
master← Head:feature/keymap-profiles📝 Commits (7)
0037aafAdd basic keymap handling874f2d1Add types for reviver and replacerfe4c45aFix reset button non-immediate component update2fd1335Fix electron only API callscdef64bAdd keymap updating instead of menu template3725b2eDo not allow null in keymap preferences5d98977Add support for global search modal toggle📊 Changes
14 files changed (+1191 additions, -347 deletions)
View changed files
📝
src/cloud/components/atoms/EditableInput.tsx(+1 -1)➕
src/components/PreferencesModal/KeymapTab.tsx(+110 -0)📝
src/components/PreferencesModal/PreferencesModal.tsx(+13 -4)📝
src/components/atoms/CodeEditor.tsx(+19 -7)📝
src/components/atoms/CustomizedCodeEditor.tsx(+2 -1)➕
src/components/atoms/KeymapItemSection.tsx(+231 -0)📝
src/components/organisms/NoteStorageNavigator.tsx(+13 -0)📝
src/electron/index.ts(+32 -3)📝
src/electron/menu.ts(+335 -321)📝
src/lib/electronOnly.ts(+3 -0)➕
src/lib/keymap.ts(+214 -0)📝
src/lib/preferences.ts(+208 -9)📝
src/locales/enUS.ts(+3 -0)📝
static/main-preload.js(+7 -1)📄 Description
Add basic keymap handling (#752)
How it looks now:

Error handling on duplicate shortcut key:

Demo how it behaves:
https://user-images.githubusercontent.com/18196945/113772151-99ce8f00-9724-11eb-8057-89393119efce.mp4
Todo:
Test:
In dev version
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.