Proposal: Improve Notification colour contrast (a11y)
See original GitHub issueThe problem
The Notification component currently fail WCAG colour contrast accessibility guidelines:
Notification with success
intent:
Element has insufficient color contrast of 2.99 (foreground color: #ffffff, background color: #0baa75, font size: 10.5pt (14px), font weight: bold). Expected contrast ratio of 4.5:1
Notification with warning
intent:
Element has insufficient color contrast of 2.85 (foreground color: #ffffff, background color: #db8500, font size: 10.5pt (14px), font weight: bold). Expected contrast ratio of 4.5:1
The proposed solution
I’d like to propose some changes to these components to make sure we can consistently meet accessibility guidelines by not setting text on a solid-colour background.
Here are some mockups of what I’d like to implement:
Success
Warning
Error
Issue Analytics
- State:
- Created 3 years ago
- Reactions:3
- Comments:11 (11 by maintainers)
looks really good! now if you compare the old notification to it, the old one look super outdated and wrong so 100x for the proposal
Do it 🤠