🎨 Palette: Add tooltip wrapper for disabled bulk action buttons - #288
🎨 Palette: Add tooltip wrapper for disabled bulk action buttons#288mapleleaflatte03 wants to merge 1 commit into
Conversation
Co-authored-by: mapleleaflatte03 <240846662+mapleleaflatte03@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 a
spanwrapper around the disabled bulk action buttons intrust-ops.htmlto provide a nativetitletooltip explaining why the button is disabled. Thetabindexandtitleattributes on the wrapper are dynamically removed viameridian.jswhen the button becomes enabled.🎯 Why: Native
titleattributes do not work on disabled buttons because they do not capture pointer events. Without this explanation, users might not know why the bulk action buttons are unavailable.📸 Before/After: Before, hovering the disabled buttons did nothing. After, hovering them shows a tooltip like "Select at least one item to approve". When an item is selected, the tooltip is removed to avoid redundant information.
♿ Accessibility: Added
tabindex="0"to the wrapper when the button is disabled so that keyboard users can access the tooltip explaining the disabled state. Thetabindexis dynamically removed when the button is enabled to prevent a confusing double tab stop (one for the wrapper, one for the button).PR created automatically by Jules for task 10586542972535773107 started by @mapleleaflatte03