Final component verification review for Expandables component#471
Final component verification review for Expandables component#471flacoman91 merged 4 commits intomainfrom
Conversation
|
billhimmelsbach
left a comment
There was a problem hiding this comment.
Looks really great! 🎉
I did notice that the aria-labels on the icons are a little off for the + and - buttons.
Here's the DS with a descriptive label (show/hide) and the role and aria-label are on the span not the svg (the svg is set to aria-hidden=true so the screen reader doesn't pick it up twice):

Here's the implementation in this PR with the name of the icon as the label and it's on the SVG not the span. So if I'm using a screen reader, I'd just hear the name of the icon instead of it's purpose (DS: show/hide)
Thanks @billhimmelsbach good catch with the icons and aria-labels! |
|
@flacoman91 |
flacoman91
left a comment
There was a problem hiding this comment.
looks good. I can approve, but it won't do anything since I last made changes to the code.
billhimmelsbach
left a comment
There was a problem hiding this comment.
Looks good to me! Thanks for the fix.

Verify Expandable component to be used in the component library
Context
https://cfpb.github.io/design-system/components/expandables
Verification checklist
1. Verify the CFPB Design System (React) component against the CFPB Design System
For existing components
2. Run accessibility checks
3. Verify component unit tests
yarn vitest Button)4. Prepare a PR for Code Review
Final component verification review for XXXX component5. Conduct Code PR review
Update Expandable.stories.tsx #447
fix: [Expandable] Correct icon color #391
Add Summary component and fixing Accordion #450
6. Conduct Component Verification PR review
DrafttoVerifiedin Storybook7. Verify component
Screenshot
Tagging
@flacoman91 @virginiacc @billhimmelsbach