Skip to content

PRXT - fix: improve DateAccordion accessibility#1405

Closed
punk6529 wants to merge 1 commit intomainfrom
codex/replace-div-with-button-in-dateaccordion
Closed

PRXT - fix: improve DateAccordion accessibility#1405
punk6529 wants to merge 1 commit intomainfrom
codex/replace-div-with-button-in-dateaccordion

Conversation

@punk6529
Copy link
Copy Markdown
Contributor

Summary

  • replace the DateAccordion header div with a semantic button that spans the full width and carries aria attributes
  • generate a stable id via useId to wire the button aria-controls to the animated panel
  • update the DateAccordion unit tests to cover the new button behavior and aria metadata

Testing

  • npx jest --runTestsByPath tests/components/common/DateAccordion.test.tsx --watchAll=false --coverage=false --silent
  • npm run lint
  • npm run type-check (fails: existing TypeScript errors across unrelated test files)

https://chatgpt.com/codex/tasks/task_e_68c93fc96ce08321849fc9d5d955c63d

Signed-off-by: OpenAI Assistant <0+openaiassistant@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

@prxt6529 prxt6529 changed the title fix: improve DateAccordion accessibility PRXT - fix: improve DateAccordion accessibility Oct 7, 2025
@prxt6529
Copy link
Copy Markdown
Collaborator

prxt6529 commented Oct 7, 2025

Will be done in #1516

@prxt6529 prxt6529 closed this Oct 7, 2025
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