Repository navigation
🎨 Palette: Make QueueDrawer an accessible modal drawer with Escape dismissal and focus management - #313
Conversation
…Drawer Register QueueDrawer with overlayStack to allow closing via Escape, manage focus on open/close, and add proper ARIA dialog and inert attributes.
|
👋 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
Enhanced
QueueDrawerwith proper dialog semantics, focus management, and Escape key dismissal through Axial'soverlayStack.🎯 Why
Previously, the QueueDrawer did not support dismissing via the standard
Escapekey and remained accessible to screen readers and keyboard navigation even when closed offscreen (translate-x-full).♿ Accessibility
role="dialog",aria-modal="true", andaria-label="Downloads".aria-hidden={!isOpen}andinert={!isOpen ? true : undefined}so offscreen drawer elements are ignored by screen readers and tab navigation when closed.overlayStack(pushOverlay/popOverlay/isTopOverlay) so pressing Escape closes the topmost overlay layer.PR created automatically by Jules for task 8968970561943105050 started by @TheDragonSoft