-
Notifications
You must be signed in to change notification settings - Fork 5
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
chore(DS): twitter icon now changes to x icon, deps(DS): Updated DS packages -- Including Icons #361
Conversation
✅ Deploy Preview for cfpb-design-system-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Thanks for working on this! I had a couple comments about the defaults.
@billhimmelsbach Removed the default |
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.
Looking good! One small NIT if you want to consider it.
…tem-react into 360-accessibility-alerts
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!
closes #588 closes #567 ## Changes - enhancement(Upload) - error alerts receive `aria-live` polite - enhancement(Footer) - Twitter icon ---> X icon Relies on cfpb/design-system-react#361 being merged in to receive the cfpb icon update
closes #362
Changes
accessibility(Alert, FieldLeveLAlert): adds aria-live region handlingScreenshot
Related: cfpb/sbl-frontend#567