Skip to content
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

Gallery icon label issue when using screen reader #1186

Open
cbaksik opened this issue Oct 29, 2024 · 2 comments
Open

Gallery icon label issue when using screen reader #1186

cbaksik opened this issue Oct 29, 2024 · 2 comments

Comments

@cbaksik
Copy link

cbaksik commented Oct 29, 2024

UV version:

I'm submitting a:

  • [X ] bug report

Current behavior:
When using a screen reader the gallery icon reads as "gallery" no matter what view is being used (full view versus gallery view). For example, if you are in full view, the gallery icon has label "gallery. When you are in gallery view the gallergy icon label is still "gallery" even though you are already in the gallery and using the button in this state will close the glallery.

Expected behavior:
When the gallery view is open, there should be a different label associated with the icon to mean "full view" instead of "gallery."

Steps to reproduce:
In full view, click on the gallery icon. While in gallery view, the label of the icon is "gallery" even though you are already in gallery view.

Other information:
I made a screen recording to demo specifically the icon in question. I narrate how the label does not change in the tooltip (same problem for screen reader): https://harvard.zoom.us/rec/share/UbI26dutHZpsloLIAYVhUjuxzxsBtTBUmQZrsC2XwLHdnXCm8yp9OBoJvEopEwqG.Mut5womckBc9dONU?startTime=1728056803000

@demiankatz
Copy link
Contributor

Thanks, @cbaksik. I'm not sure it would be entirely clear to have the Gallery icon label change to "Full View" when the Gallery is open, because I think it might be confusing in relation to the adjacent "Single page view" / "Two page view" icons. The Gallery really operates independently of those other controls as a toggle.

I'd suggest one of two possible approaches:

1.) Have the Gallery icon label switch between "Open Gallery view" and "Close Gallery view"

2.) Change the Gallery icon label to "Toggle Gallery view" (possibly with a parenthetical indicator or some other aria attributes to indicate the current state of the gallery)

What do you think?

@cbaksik
Copy link
Author

cbaksik commented Oct 29, 2024

Thanks for your quick response. Option 1 sounds perfect, and better than what I suggested :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants