Skip to content

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

Merged
merged 13 commits into from
Jan 14, 2025

Conversation

Computerdores
Copy link
Collaborator

@Computerdores Computerdores commented Aug 27, 2024

Closes #405.

@Computerdores Computerdores changed the title Keyboard nav improvements Add Tag Modal: Keyboard nav improvements Aug 27, 2024
@CyanVoxel CyanVoxel added Type: QoL A quality of life (QoL) enhancement or suggestion Type: UI/UX User interface and/or user experience Priority: Low Doesn't require immediate attention Status: Review Needed A review of this is needed Type: Enhancement New feature or request labels Aug 27, 2024
Copy link

@eivl eivl left a 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.

@CyanVoxel CyanVoxel removed the Status: Review Needed A review of this is needed label Sep 6, 2024
@Computerdores Computerdores force-pushed the keyboard_nav_improvements branch from 155a5ac to fd28e10 Compare September 20, 2024 14:31
@Computerdores Computerdores changed the base branch from Alpha-v9.4 to main September 20, 2024 14:31
@Computerdores
Copy link
Collaborator Author

I made no functional changes, only rebased onto main

@CyanVoxel
Copy link
Member

I would love to pull this, but unfortunately there was some non-final UI changes to aliases that were attached to #596. If the alias UI gets reverted before #534 is finalized then I'd really like to pull this on top of the old UI.

@CyanVoxel CyanVoxel added the Status: Blocked This issue or pull request is awaiting the outcome of another issue or pull request label Dec 5, 2024
@Computerdores
Copy link
Collaborator Author

As far as I am concerned I could also adapt this to the new UI once that is on main, up to you

@CyanVoxel
Copy link
Member

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

@Computerdores
Copy link
Collaborator Author

Computerdores commented Dec 21, 2024

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 1

image

Idea 2

image

What's your take on this?

PS: For now I am changing it to Idea 1 because the current state is worse than both

@CyanVoxel
Copy link
Member

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.

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.

@CyanVoxel CyanVoxel added this to the Alpha v9.5 (Post-SQL) milestone Dec 22, 2024
@CyanVoxel CyanVoxel removed the Status: Blocked This issue or pull request is awaiting the outcome of another issue or pull request label Dec 23, 2024
@CyanVoxel
Copy link
Member

Removing the "blocked" status now that #534 has been merged

@Computerdores Computerdores added the Status: Review Needed A review of this is needed label Dec 23, 2024
@CyanVoxel CyanVoxel added Priority: Medium An issue that shouldn't be be saved for last and removed Priority: Low Doesn't require immediate attention labels Jan 3, 2025
@CyanVoxel CyanVoxel added Status: Mergeable The code is ready to be merged and removed Status: Review Needed A review of this is needed labels Jan 13, 2025
@CyanVoxel
Copy link
Member

Thank you so much for your work on this!

@CyanVoxel CyanVoxel merged commit 0cdb1a8 into TagStudioDev:main Jan 14, 2025
5 checks passed
@CyanVoxel CyanVoxel removed the Status: Mergeable The code is ready to be merged label Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Medium An issue that shouldn't be be saved for last Type: Enhancement New feature or request Type: QoL A quality of life (QoL) enhancement or suggestion Type: UI/UX User interface and/or user experience
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Better Keyboard Navigation For Add Tag Modal
3 participants