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
Accessibility updates to alert for toast alerts #3519
Conversation
PatternFly-React preview: https://patternfly-react-pr-3519.surge.sh |
9f6fe5a
to
5e4862b
Compare
f059db8
to
d8ba6a7
Compare
Codecov Report
@@ Coverage Diff @@
## master #3519 +/- ##
=========================================
Coverage ? 67.09%
=========================================
Files ? 903
Lines ? 25486
Branches ? 2260
=========================================
Hits ? 17100
Misses ? 7344
Partials ? 1042
Continue to review full report at Codecov.
|
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.
Thanks @jessiehuff ! 🎖
Your changes have been released in:
Thanks for your contribution! 🎉 |
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.
You are missing integration test and you need to add the new prop to the demo.
@@ -57,6 +60,7 @@ const Alert: React.FunctionComponent<AlertProps & InjectedOuiaProps> = ({ | |||
const customClassName = css( | |||
styles.alert, | |||
isInline && styles.modifiers.inline, | |||
isToast && 'pf-m-live', |
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.
Why are you using the class name here instead of react-styles?
What: Closes #1421