Skip to content

fix(a11y): accessible names for help buttons and scope status icons - #3558

Merged
setchy merged 1 commit into
mainfrom
fix/accessible-help-and-scope-icons
Oct 28, 2026
Merged

setchy merged 1 commit into
mainfrom
fix/accessible-help-and-scope-icons

Conversation

@setchy

@setchy setchy commented Oct 28, 2026

Copy link
Copy Markdown
Member

Summary

  • Help buttons: screen readers announced the internal id (e.g. "tooltip-search"). HelpPopover now takes a label and reads "Help: ".
  • Scope status icons: were aria-hidden, so granted/missing was conveyed by colour alone. ScopeStatusIcon now has an accessible name (e.g. "repo missing"); the withTestId prop is removed.
  • Tests query these by role instead of test ID.

Follow-up to #3510 / #3557.

Help buttons announced their internal id (e.g. "tooltip-search"); they now
read "Help: <label>". Scope status icons were hidden from assistive tech,
so granted/missing was conveyed by colour alone; they now carry a label such
as "repo missing". Tests query by role instead of test ID.
@setchy
setchy requested a review from afonsojramos as a code owner October 28, 2026 23:20
@github-actions github-actions Bot added the bug Something isn't working label Oct 28, 2026
@sonarqubecloud

Copy link
Copy Markdown

@setchy
setchy merged commit 3866936 into main Oct 28, 2026
19 checks passed
@setchy
setchy deleted the fix/accessible-help-and-scope-icons branch October 28, 2026 23:25
@github-actions github-actions Bot added this to the Release 8.0.0 milestone Oct 28, 2026
setchy added a commit that referenced this pull request Oct 29, 2026
Resolve conflicts with #3449, #3557, #3558 and the metric pill changes:
keep main's structure (device-flow wrapper, custom OAuth app, removed
test IDs, chevron directions) and wrap its strings for translation.
Translate the screens main added since this branch was cut
(DeviceFlowAuthorize, LoginWithCustomOAuthApp, device-flow start errors,
ScopeStatusIcon, HelpPopover) and regenerate en.json.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Development

Successfully merging this pull request may close these issues.

1 participant