🎨 Palette: Add tooltips for disabled bulk actions - #297
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. |
Co-authored-by: mapleleaflatte03 <240846662+mapleleaflatte03@users.noreply.github.com>
Co-authored-by: mapleleaflatte03 <240846662+mapleleaflatte03@users.noreply.github.com>
Co-authored-by: mapleleaflatte03 <240846662+mapleleaflatte03@users.noreply.github.com>
💡 What
Added tooltips to the bulk action buttons (approve, stale, revoke, unresolved) in the Trust Ops queue when they are disabled.
🎯 Why
Disabled buttons swallow pointer events by default and don't show native title tooltips. When users see disabled buttons without knowing why, it creates a confusing UX. Adding a wrapper with a tooltip explains the required state ("Select one or more items first").
📸 Before/After
Before: The buttons were just greyed out, with no explanation on hover or focus.
After: When disabled, hovering over or focusing on the buttons shows a tooltip "Select one or more items first". When enabled, the tooltip goes away.
♿ Accessibility
Wrapped disabled buttons in a
spanwithtabindex="0", allowing keyboard-only users to focus the wrapper and have screen readers announce thetitle. Thistabindexis dynamically removed when the buttons become enabled to prevent a double-tab-stop issue.PR created automatically by Jules for task 8529921724311024066 started by @mapleleaflatte03