-
-
Notifications
You must be signed in to change notification settings - Fork 388
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
fix: better align access keys with the guidelines #843
base: main
Are you sure you want to change the base?
Conversation
While looking into adding mnemonics to the search bar in the main window, I have discovered that adding them to a We might consider adding a shortcut key to focus on the text box (Ctrl + F, for instance) and signal this in the placeholder text (‘Search Entries ({shortcut})’), but that is out of scope for this PR. Alternatively, we could add a buddied |
I apologise; I realised I have made this PR too soon, as there are still more commits I wish to make. I will reopen this when I am done. |
I have made all the modifications I wanted and updated the PR to reflect that. It is now ready for review. |
Summary
Resolves #678.
Based on Microsoft's guidelines, I have…
Unresolved issues:
Tasks Completed