🎨 Palette: Improve table header sorting accessibility#44
Conversation
- Added role="button", tabindex="0", and aria-label to table header sorting wrappers. - Implemented keyboard support (Enter and Space) for triggering sort. - Dynamically update aria-sort attribute on th elements. - Added visual focus indicators using Tailwind ring utilities. - Normalized line endings to LF for app/templates/shared/list_base.html. 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
Improved the accessibility of the table sorting functionality in the shared list base template. Previously, sorting was only accessible via mouse clicks on headers. This change introduces proper ARIA roles, labels, and keyboard interaction support, along with visual focus feedback for keyboard users.
PR created automatically by Jules for task 17960063028087905169 started by @Woschj