Skip to content

Final component verification review for Expandables component#471

Merged
flacoman91 merged 4 commits intomainfrom
natalia-fitzgerald-patch-7
Feb 5, 2026
Merged

Final component verification review for Expandables component#471
flacoman91 merged 4 commits intomainfrom
natalia-fitzgerald-patch-7

Conversation

@natalia-fitzgerald
Copy link
Copy Markdown
Collaborator

@natalia-fitzgerald natalia-fitzgerald commented Feb 5, 2026

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

Design System Design System in React
Image Image
  • Check to see if the applied CSS styles match the specs in the DS if applicable (message @natalia-fitzgerald if it does not)*
  • Manual visual review has been conducted and component has passed this review

2. Run accessibility checks

  • Component is keyboard-friendly (navigable with tab, space, enter, arrow keys, etc.)
  • Component does not introduce new errors or warnings in WAVE
  • Component is screen reader friendly (screen reader testing demo)
    • Is all the meaningful visual information and text of the component conveyed by the screen reader? (text, non-decorative image descriptions, etc.)
    • When interacting with the component using a screen reader, do you have all the information you need to use it? (selected vs unselected, button vs link, expanded vs collapsed, etc.)
    • Does the component have similar screen reader behavior as the sample components in WCAG, the CFPB design system, WebAIM, or similar accessibility examples?
  • For reference: CFPB manual web accessibility audit

3. Verify component unit tests

  • Verify component unit tests are implemented and passing - 85% or greater (ex: yarn vitest Button)
Unit tests
Image

4. Prepare a PR for Code Review

  • Create a PR with the name: Final component verification review for XXXX component
  • Copy this verification checklist that's almost complete into the PR description

5. Conduct Code PR review

6. Conduct Component Verification PR review

  • Apply GitHub label "component-verification"
  • Change the component's status from Draft to Verified in Storybook
  • Add a DSR contributor as a reviewer (A designer and a developer must review)
  • Ping DSR contributors in Slack in the Design System in React channel when ready

7. Verify component

  • Merge when design and dev review completed to finish component verification 🎉

Screenshot

Screenshot 2026-02-05 at 2 01 33 PM Screenshot 2026-02-05 at 2 01 42 PM

Tagging
@flacoman91 @virginiacc @billhimmelsbach

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 5, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-02-05 23:02 UTC

@natalia-fitzgerald natalia-fitzgerald marked this pull request as ready for review February 5, 2026 19:03
Copy link
Copy Markdown
Collaborator

@billhimmelsbach billhimmelsbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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):
Image

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)

Image

@flacoman91
Copy link
Copy Markdown
Collaborator

The sorting of the Expandables is appearing at the bottom for some reason.

I had to CMD+F to find it

Screenshot 2026-02-05 at 12 27 20 PM

@flacoman91
Copy link
Copy Markdown
Collaborator

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):

Thanks @billhimmelsbach good catch with the icons and aria-labels!

@natalia-fitzgerald
Copy link
Copy Markdown
Collaborator Author

@flacoman91
It looks like you fixed the ordering. Nice work!

Copy link
Copy Markdown
Collaborator

@flacoman91 flacoman91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good. I can approve, but it won't do anything since I last made changes to the code.

Copy link
Copy Markdown
Collaborator

@billhimmelsbach billhimmelsbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Thanks for the fix.

@flacoman91 flacoman91 merged commit ade27eb into main Feb 5, 2026
4 checks passed
@flacoman91 flacoman91 deleted the natalia-fitzgerald-patch-7 branch February 5, 2026 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants