Skip to content

fix: improve category select accessibility - #4836

Open
Atmuri-SatyaPrakash wants to merge 5 commits into
ONEARMY:masterfrom
Atmuri-SatyaPrakash:fix/4827-category-select-accessibility
Open

fix: improve category select accessibility#4836
Atmuri-SatyaPrakash wants to merge 5 commits into
ONEARMY:masterfrom
Atmuri-SatyaPrakash:fix/4827-category-select-accessibility

Conversation

@Atmuri-SatyaPrakash

Copy link
Copy Markdown

Summary

  • replace the legacy CategoryHorizonalList with the new UI component
  • add accessible aria-pressed state for category selection
  • add visible selected-state styling
  • add Storybook coverage for the new component

Validation

  • TypeScript check passes
  • Prettier check passes
  • 58/58 test files pass
  • 327/327 tests pass
  • verified category selection and deselection in Storybook

@cypress

cypress Bot commented Aug 14, 2026

Copy link
Copy Markdown

onearmy-community-platform    Run #9887

Run Properties:  status check passed Passed #9887  •  git commit 2049969f15: fix: preserve category item test selector
Project onearmy-community-platform
Branch Review pull/4836
Run status status check passed Passed #9887
Run duration 34m 52s
Commit git commit 2049969f15: fix: preserve category item test selector
Committer unknown
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 113
View all changes introduced in this branch ↗︎

@mariojsnunes mariojsnunes added the Review allow-preview ✅ Has received manual check for malicious code and can be safely built for preview label Aug 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review allow-preview ✅ Has received manual check for malicious code and can be safely built for preview

Projects

Status: No status
Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants