kbar icon indicating copy to clipboard operation
kbar copied to clipboard

Updating tiny keys, adding shortcut tests

Open npfitz opened this issue 8 months ago • 5 comments

Fixes https://github.com/timc1/kbar/issues/374, and adds some unit tests around shortcuts. Admittedly, the shortcut cases are pretty sparse. I'd be happy to add any new cases or update if it's apropriate.

Alternatively, is there a reason we don't just import tinyKeys as a dependency? I'd be happy to update the PR to do this as well, rather than copy the source for tinyKeys into the repo

npfitz avatar Apr 21 '25 15:04 npfitz

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kbar-bpez ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 21, 2025 3:21pm

vercel[bot] avatar Apr 21 '25 15:04 vercel[bot]

@timc1 Any thoughts on the above changes?

npfitz avatar Apr 22 '25 11:04 npfitz

+1 to adding tinykeys as a package instead of adding the source code

edit, I did a patch on my project that simply used the tinyKeys package and it fixed it for me!

mrdjohnson avatar Aug 01 '25 06:08 mrdjohnson

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.

stale[bot] avatar Dec 03 '25 10:12 stale[bot]

Would be really helpful for this fix to be merged - I'm also seeing the same issue

pete-woods avatar Dec 04 '25 21:12 pete-woods