-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
fix(notification): fix low-contrast close button focus color #5793
fix(notification): fix low-contrast close button focus color #5793
Conversation
Deploy preview for carbon-components-react ready! Built with commit bb6b890 https://deploy-preview-5793--carbon-components-react.netlify.com |
Deploy preview for carbon-elements ready! Built with commit bb6b890 |
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 am going to approve this because this logic works correctly.
Eventually we do need to update our low contrast notifications in the dark themes, but for now it is still blocked by this issue: #4282
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 given the related blocking issue
Closes #5737
Adds normal focus styles to low-contrast
Notification
Changelog
New
Testing / Reviewing
Focus on the close button in low-contrast mode and ensure the focus indicator is visible