🎨 Palette: Enhance table sorting accessibility and search UI#35
🎨 Palette: Enhance table sorting accessibility and search UI#35
Conversation
This change implements a micro-UX improvement for the inventory list views: - Added keyboard navigation support (Enter/Space) and proper ARIA roles/attributes to table sorting headers. - Added visual focus indicators to sortable headers for better keyboard accessibility. - Updated the search bar to use modern DaisyUI 'join' layout for improved visual grouping. - Fixed a syntax error in base.html (unbalanced HTML tags for QuickScan trigger). ♿ Accessibility: - Added role="button", tabindex="0", and aria-label to sortable headers. - Implemented aria-sort state updates for screen reader feedback. - Fixed stray button tag to ensure a valid DOM tree. 🎯 Why: Users relying on keyboard navigation were previously unable to sort tables. Screen reader users lacked feedback on the current sort state. The search UI was using deprecated DaisyUI components. Co-authored-by: Woschj <81321922+Woschj@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. |
1 participant
Implemented accessible table sorting and modernized the search UI in
shared/list_base.html. Also fixed a syntax error inbase.html. The changes focus on keyboard accessibility and screen reader support while maintaining a minimal and clean diff by respecting the original CRLF line endings of the templates.PR created automatically by Jules for task 7323283206684647114 started by @Woschj