feat: add toggle shortcut option and support tinykey keybindings

[avigoldman]

Closes #55 and #121

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/timc/kbar/GVRqXR62hYeBKrukCjMJ6MqU3gpZ
✅ Preview: https://kbar-git-fork-avigoldman-issue-55-timc.vercel.app

[avigoldman]

There seems to be an issue with tinykeys not supporting special characters like ?. I did PR as a first pass to fix that issue here. jamiebuilds/tinykeys#131

[stale]

Hey! This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[smhutch]

@avigoldman @timc1 is there anything I could do to help with this PR? I'm working on a code editor, meaning we can't really use single character keyboard shortcuts (since we focus the editor in most cases 😄)

I would be happy to dedicate some time towards adding support for combinations like CMD + 1 or Ctrl + x etc. Any particular place I could help out here?

[timc1]

Hey @smhutch, we can get rid of the event.metaKey check here in the current implementation to get cmd/ctrl to work properly; however, we would need to add a few additional safeguards against registering cmd/ctrl+k.

I think it should be pretty straight forward; we originally added the event.metaKey as more of an easy solution. If you want to take a stab at removing it that would be great – I'll look into a solution as well tomorrow.

[stale]

Hey! This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[timc1]

Closed as part of #185