Skip to content

fix(frontend): add accessible currency input semantics - #592

Merged
dark-sarge merged 1 commit into
arflexx:mainfrom
Sammyzyandev:fix/issue-457
Oct 26, 2026
Merged

dark-sarge merged 1 commit into
arflexx:mainfrom
Sammyzyandev:fix/issue-457

Conversation

@Sammyzyandev

Copy link
Copy Markdown
Contributor

Summary

  • Expose CurrencyInput as a named spinbutton with its current Naira value and applicable minimum/maximum.
  • Mark validation errors as assertive live updates while preserving the visible keyboard focus ring.
  • Add coverage for the input's accessible name, bounds, error announcement, and axe checks.

Validation

  • npm test -- --runInBand components/CurrencyInput.test.tsx — passed (8 tests), including axe.
  • npm run lint (TypeScript check) reports existing failures in unrelated metadata, formatting, and config files; no CurrencyInput errors were reported.
  • git diff --check and full added-line PolinRider scan — passed.

Closes #457

@dark-sarge
dark-sarge merged commit 2a91010 into arflexx:main Oct 26, 2026
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.

[Frontend] - Add ARIA Labels to Currency Input Component

2 participants