🎨 Palette: Fix IconButton minimum touch target accessibility#690
🎨 Palette: Fix IconButton minimum touch target accessibility#690yuga-hashimoto wants to merge 1 commit into
Conversation
Co-authored-by: yuga-hashimoto <74749461+yuga-hashimoto@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: Removed explicitly defined small size modifiers (
Modifier.size(24.dp)) fromIconButtonwrappers and moved them to the innerIconelements instead across the app.🎯 Why: Setting a small size on the
IconButtondirectly overrides its built-in accessible padding. This results in the touch target being shrunk to 24x24dp, which is extremely difficult to tap reliably and fails accessibility standards.📸 Before/After:
Before: Touch bounding box rigidly snapped to the 24dp icon edge.
After: Visual icon remains 24dp, but touch target correctly expands to >=48dp.
♿ Accessibility: This change restores the Material Design standard 48x48dp minimum touch target for these buttons, significantly improving usability for users with motor impairments or those on smaller devices.
PR created automatically by Jules for task 16611886130857712751 started by @yuga-hashimoto