馃帹 Palette: Add aria-controls to change request accordion - #339
馃帹 Palette: Add aria-controls to change request accordion#339brycejohnson1417 wants to merge 1 commit into
Conversation
Co-authored-by: brycejohnson1417 <257422776+brycejohnson1417@users.noreply.github.com>
|
馃憢 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 馃憖 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
1 participant
What: Added
aria-controlsto the toggle buttons in the ChangeRequestList component and wrapped the conditionally rendered content in a persistent container with the corresponding ID.Why: When an accordion's content is conditionally rendered, the
aria-controlsID must exist in the DOM even when collapsed to avoid breaking screen reader associations.Before/After: No visual change, structural HTML accessibility improvement.
Accessibility: Ensures screen readers correctly associate the toggle buttons with the expandable content area in both expanded and collapsed states.
PR created automatically by Jules for task 203978469355574701 started by @brycejohnson1417