🎨 Palette: Enhance Search UX and Fix Accessibility#47
Conversation
- Refactor QuickScan trigger to semantic button for better accessibility. - Modernize search bar using DaisyUI 'join' component. - Add dynamic 'Clear Search' button to the search input. - Add 'No results found' feedback row to data tables. - Update JavaScript for smoother filtering and state management. 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. |
💡 What
This PR introduces micro-UX improvements to the Scandy search interface and fixes an accessibility regression in the base layout.
🎯 Why
div/buttontag combination, which corrupted the accessibility tree.♿ Accessibility
quickScanTriggerto a semantic<button>element, which natively handles keyboard navigation and ARIA roles.aria-labelandtitleattributes to the new "Clear Search" button.aria-live="polite"indirectly via the "No results" row to ensure screen readers are aware of empty states.📸 UI Changes
joincomponent with integrated borders and focus rings.PR created automatically by Jules for task 11731302769807672902 started by @Woschj