Skip to content

a11y: add ARIA labels to sample buttons; improve live region with ari…#133

Merged
Deepika14145 merged 1 commit intoDeepika14145:mainfrom
Adithyakp86:main
Oct 20, 2025
Merged

a11y: add ARIA labels to sample buttons; improve live region with ari…#133
Deepika14145 merged 1 commit intoDeepika14145:mainfrom
Adithyakp86:main

Conversation

@Adithyakp86
Copy link
Contributor

…a-busy and focus management on result updates
#82

What changed
templates/index.html
Added aria-labels to example buttons in the “Try these examples” section.
Made #prediction-result focusable (tabindex="-1") and added aria-busy="false".
Set #result-message as a live region with aria-live="polite" and aria-atomic="true".
templates/index_i18n.html
Added aria-labels to Example 1–3 buttons.
Made #prediction-result focusable (tabindex="-1") and added aria-busy="false".
Set #result-message as a live region with aria-live="polite" and aria-atomic="true".

static/script.js
In displayResult(...), now toggles aria-busy to true on loading and false after update.
Moves focus to #prediction-result after non-loading updates to prompt screen reader announcement.

…a-busy and focus management on result updates
@vercel
Copy link

vercel bot commented Oct 20, 2025

@Adithyakp86 is attempting to deploy a commit to the deepika's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Deepika14145 Deepika14145 merged commit 9e93db4 into Deepika14145:main Oct 20, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants