-
-
Notifications
You must be signed in to change notification settings - Fork 394
Add Tag Modal: Keyboard nav improvements #407
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
Add Tag Modal: Keyboard nav improvements #407
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic change, this introduces the add tag modal with the expected behaviour.
155a5ac
to
fd28e10
Compare
I made no functional changes, only rebased onto main |
As far as I am concerned I could also adapt this to the new UI once that is on main, up to you |
If #534 develops and gets merged then that would be fantastic! If not or if the old UI is restored in the meantime, then I'll likely pull this as-is |
I am reworking the keyboard navigation rn and would like some input regarding how the Tab Focus should move through the window. Main question is how to handle the lists of Aliases and Parent Tags. The first idea has the advantage of being "what you see is what you get", while the second one has the advantage of always requiring the same number of tabs to go to Alias Add, Subtag Add, Color, Cancel and Save: I personally lean towards the second idea as it would make the save and cancel buttons more easily reachable, but the first idea with shortcuts for color, save, cancel and the add buttons (save and cancel at the very least) would also be an option imo. Idea 1Idea 2What's your take on this? PS: For now I am changing it to Idea 1 because the current state is worse than both |
I think I prefer the second idea as well, and it would pair nicely with the arrow keys being used to navigate back up into the pre-existing alias list so it stays accessible. I also know there's been some confusion around this, but the alias UI currently in the program is not permanent and was never supposed to be added - the "final" alias UI is due to be added in #534, however that shouldn't change the general idea of the navigation here, just the implementation specifics. |
Removing the "blocked" status now that #534 has been merged |
…or code deduplication" This reverts commit fac589b.
Thank you so much for your work on this! |
Closes #405.