Skip to content

Replace aria-valuetext with aria-description for improved accessibility#1286

Merged
ronalduQualabs merged 1 commit intomainfrom
fix/switch-aria-valuetext-to-aria-description
Apr 9, 2026
Merged

Replace aria-valuetext with aria-description for improved accessibility#1286
ronalduQualabs merged 1 commit intomainfrom
fix/switch-aria-valuetext-to-aria-description

Conversation

@ronalduQualabs
Copy link
Copy Markdown
Contributor

@ronalduQualabs ronalduQualabs commented Apr 8, 2026

Replace aria-value with aria-description for improved accessibility resolve 1280


Note

Low Risk
Low risk: small, localized accessibility attribute change on media-time-display with no behavioral logic changes beyond what ARIA attribute is updated.

Overview
Updates media-time-display to stop writing time announcements to aria-valuetext and instead maintain an aria-description that reflects current/remaining time and optional duration, including an explicit "video not loaded" description when time data is unavailable.

Renames the helper from updateAriaValueText to updateAriaDescription and wires the new function into the component update() path so the description stays in sync with frequent time updates.

Reviewed by Cursor Bugbot for commit 94254f5. Bugbot is set up for automated code reviews on this repo. Configure here.

@ronalduQualabs ronalduQualabs requested review from a team and heff as code owners April 8, 2026 17:42
@snyk-io
Copy link
Copy Markdown

snyk-io bot commented Apr 8, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 8, 2026

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

Project Deployment Actions Updated (UTC)
media-chrome Ready Ready Preview, Comment Apr 8, 2026 5:57pm
media-chrome-demo-nextjs Ready Ready Preview, Comment Apr 8, 2026 5:57pm
media-chrome-docs Ready Ready Preview, Comment Apr 8, 2026 5:57pm

Request Review

@R-Delfino95 R-Delfino95 self-assigned this Apr 9, 2026
@ronalduQualabs ronalduQualabs merged commit 3b1014b into main Apr 9, 2026
12 checks passed
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.

[A11y] ARIA attribute aria-valuetext is not allowed in media-time-display

2 participants