search

silverbulletmd / silverbullet

The hackable notebook
https://silverbullet.md
MIT License
2.06k stars 146 forks source link

Correctly skip adding default shortcuts for overriden commands #739

Closed joekrill closed 4 months ago

joekrill commented 4 months ago

A small bug was causing existing shortcuts to continue to work even when adding overrides to SETTINGS (the overrides worked as well).

Fixes #688

(Note this does not fix #689 - looks like that's caused by something different)

@zefhemel I hope you don't mind me putting up these PRs! I thought I could help fix some of these small bugs.

zefhemel commented 4 months ago

Dude, I really appreciate the help!