-
Notifications
You must be signed in to change notification settings - Fork 141
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
USWDS-Site: Added combo box accessibility tests page #2811
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@amycole501 and @mahoneycm - There are a few things I wasn't sure about for this checklist and made comments on them in the "Files Changed" tab. Let me know if they are correct or any guidance on what to update would be great!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work here @RachelCorsino! Everything looks good here. Just had a couple of questions for @amycole501 and @alex-hull.
Co-authored-by: Amy Leadem <[email protected]>
There was a problem hiding this 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!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks @RachelCorsino 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - Thanks @RachelCorsino!
@amycole501 or @alex-hull, could you review and confirm that everything appears as expected?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it looks good! Approved on my end!
title: Combo box accessibility tests | ||
type: component | ||
items: | ||
- date: NNNN-NN-NN |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Important
We need to update the changelog dates before merge
Summary
Added accessibility test page for combo box component
Related issue
Closes #2768
Preview link
Preview link:
Combo Box Component Page
Testing and review
Follow these steps: