🎨 Palette: Add missing ARIA labels to icon-only buttons#1248
🎨 Palette: Add missing ARIA labels to icon-only buttons#1248khangnghiem wants to merge 1 commit into
Conversation
Adds `aria-label`s specifically to icon-only buttons in `site/index.html` to improve accessibility for screen reader users. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@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. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
💡 What
Added
aria-labelattributes exclusively to icon-only buttons (like tool selectors, undo/redo, etc.) insite/index.html.🎯 Why
Icon-only buttons are completely opaque to screen readers without an
aria-label. This was a major accessibility gap in the web application's UI, especially for core interaction tools.📸 Before/After
<button class="ft-tool-btn" data-tool="hand"><svg>...</svg></button><button class="ft-tool-btn" data-tool="hand" aria-label="Hand"><svg>...</svg></button>♿ Accessibility
This change ensures that all primary interaction tools on the canvas (Hand, Select, Rectangle, Ellipse, Pen, Arrow, Text, Eraser, Lasso, Undo, Redo, AI Touch) are now discoverable and clearly announced to screen reader users, adhering to WCAG guidelines for non-text content. Care was taken to NOT add redundant
aria-labelsto buttons that already have visible text.PR created automatically by Jules for task 2898699611078508689 started by @khangnghiem