-
Notifications
You must be signed in to change notification settings - Fork 13
DBNotification: Add default ARIA role attributes for accessibility #4815
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: mfranzke <787658+mfranzke@users.noreply.github.com>
|
@copilot please retry |
Implementation complete! ✅ The DBNotification component now has default role attributes for accessibility:
Changes are minimal and focused - only 20 lines total across 2 files. Commit: 6271418 Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Enhances the DBNotification component with appropriate ARIA role attributes based on semantic values to improve accessibility for assistive technologies.
Changes Made
Added a utility function
getNotificationRolethat maps semantic notification types to appropriate ARIA roles:criticalandwarning→alert(urgent messages requiring immediate attention)informationalandsuccessful→status(informational messages)The role attribute is now automatically applied to the notification's article element based on the
semanticprop value.Example Usage
This implementation follows ARIA best practices and enhances screen reader support without breaking existing functionality.
Fixes #4206.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
🔭🐙🐈 Test this branch here: https://design-system.deutschebahn.com/core-web/review/copilot/fix-4206