Skip to content

Issue 1496 a11y text#1534

Merged
demiankatz merged 2 commits into
UniversalViewer:devfrom
LlGC-jop:issue_1496-a11y_text
Sep 4, 2025
Merged

Issue 1496 a11y text#1534
demiankatz merged 2 commits into
UniversalViewer:devfrom
LlGC-jop:issue_1496-a11y_text

Conversation

@LlGC-jop

@LlGC-jop LlGC-jop commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

Fixes #1496

Have restored the title attr to the collapse button.

First/Prev buttons seem to already have it in the latest dev branch, so it may have been restored at some point.

Tested with NVDA.

@vercel

vercel Bot commented Sep 4, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
universalviewer Ready Ready Preview Comment Sep 4, 2025 2:18pm

@LanieOkorodudu LanieOkorodudu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @LlGC-jop for addressing this issue. I can confirm that the label has now been restored. It looks good to merge, and we can probably close the issue as well. I appreciate your time on this!

@demiankatz demiankatz merged commit bd51a2f into UniversalViewer:dev Sep 4, 2025
4 checks passed
@demiankatz

Copy link
Copy Markdown
Contributor

Thanks for the fix, @LlGC-jop and for the confirmation/testing, @LanieOkorodudu! I've gone ahead and merged this. I'm going to leave a separate comment on #1496 before closing that issue -- see you over there.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Screen Reader Accessibility Regression: Missing Labels and Unread Buttons in v4.2.1

3 participants