-
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 radio button accessibility test page #2821
Conversation
@mahoneycm @amyleadem - I'm not seeing the accessibility tests show up on the page... Let me know if you guys notice what I'm missing to make that work! I double checked I put everything in as stated in the Creating Accessibility Checklists doc. I do see the changelog in the Latest Updates section so at least that's working. |
@RachelCorsino, to connect the pages, I believe you'll need to rename |
Update: I went ahead and updated the file name in 7a9b7f6 so I can review. |
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, @RachelCorsino. I had just a couple of questions for @alex-hull and @amycole501 below.
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.
Left a couple comments about possible wording corrections. Looks good otherwise!
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.
Thank you! Looks good to me!
…dio-button-checklist
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!
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 thought I already approved yesterday, but I'll approve again today! LGTM :)
title: Radio button 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 radio button component
Related issue
Closes #2800
Preview link
Preview link: Radio Buttons component page
Testing and review
Follow these steps: