Skip to content

fix(a11y): announce password criteria status - #217

Open
terminalchai wants to merge 1 commit into
nastaso:mainfrom
terminalchai:fix/101-password-criteria-a11y
Open

fix(a11y): announce password criteria status#217
terminalchai wants to merge 1 commit into
nastaso:mainfrom
terminalchai:fix/101-password-criteria-a11y

Conversation

@terminalchai

@terminalchai terminalchai commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Hides the password criteria status icons from assistive technology and adds screen-reader-only text that announces whether each criterion is met or not met. The visual layout and password scoring behavior are unchanged.

Why?

Screen readers currently encounter decorative icons without an explicit textual state for each password requirement.

Closes #101

How was this tested?

  • npm run check (Astro check, ESLint, and 300 tests)
  • npm run build with placeholder public Supabase environment values
  • Playwright checks at 390x844 and 1440x1000 confirmed all four criteria expose met/not met text, all criterion icons are hidden, and the status text remains visually clipped

Checklist

  • npm run build passes locally
  • npm run lint passes locally
  • npm run test passes locally
  • npm run validate is not applicable because no src/data/**/*.json files changed
  • No new third-party dependencies

Screenshots (UI changes only)

No visible UI change.

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.

a11y(PasswordStrengthMeter): announce password criteria met/not-met state to screen readers

1 participant