🎨 Palette: [UX improvement] Enhance Dialog component accessibility#222
🎨 Palette: [UX improvement] Enhance Dialog component accessibility#222ivanleekk wants to merge 1 commit into
Conversation
- Added `role="dialog"` and `aria-modal="true"` to modal container - Added `aria-hidden="true"` to backdrop - Added `aria-label="Close dialog"` and `focus-visible` styles to close button Co-authored-by: ivanleekk <84584280+ivanleekk@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
💡 What
Added essential ARIA attributes and focus styles to the
Dialogcomponent to improve its accessibility.aria-hidden="true".role="dialog"andaria-modal="true".aria-label="Close dialog"andfocus-visiblestyles for better keyboard navigation.🎯 Why
Dialogs are critical UI elements that require proper ARIA attributes to be announced correctly by screen readers. Without these attributes, screen reader users might not realize a modal has opened or be able to easily find the close button. Furthermore, icon-only buttons always need an
aria-labelfor assistive technologies. Addingfocus-visibleensures that users navigating via keyboard can clearly see when the close button has focus.📸 Before/After
(Visuals remain unchanged for standard pointer interactions, but keyboard focus ring is now visible on the close button, and screen reader announcements are significantly improved.)
♿ Accessibility
PR created automatically by Jules for task 5307551743830549429 started by @ivanleekk