-
Notifications
You must be signed in to change notification settings - Fork 770
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve wording of shortcut hotbar #354
Labels
polish
Small improvements to the experience
Comments
Perhaps we should also accept the upper case letters too? |
Sounds like a good idea, especially if you've accidentally turned on caps-lock. |
Partially resolved by #363 |
petebacondarwin
added a commit
to petebacondarwin/wrangler2
that referenced
this issue
Apr 3, 2022
petebacondarwin
added a commit
that referenced
this issue
Apr 4, 2022
Repository owner
moved this from In Review
to Done
in workers-sdk
Apr 4, 2022
Merged
This was referenced Nov 23, 2023
This was referenced Dec 1, 2023
This was referenced Jan 10, 2024
This was referenced Jan 17, 2024
This was referenced Jan 25, 2024
This was referenced Feb 1, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Current shortcuts:
These shortcuts imply a capital letter, but only lowercase letters work. Instead, we should use the lowercase letters, but either color or bold them to make it more prominent.
I also think we should hide
[s]
since it's experimental.The text was updated successfully, but these errors were encountered: