🎨 Palette: [UX improvement] Add ARIA labels to NewAgentModal - #86
🎨 Palette: [UX improvement] Add ARIA labels to NewAgentModal#86google-labs-jules[bot] wants to merge 1 commit into
Conversation
Added explicit ARIA labels and roles to the NewAgentModal component to improve accessibility for screen readers. - Converted vendor selection to a proper `radiogroup` with `radio` roles. - Added explicit `aria-label` to the Repo and Branch inputs. - Associated the Task label with its Textarea using `htmlFor` and `id`.
|
👋 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. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
💡 What
Added explicit ARIA labels, roles, and label associations to form fields in the
NewAgentModalcomponent.🎯 Why
Inputs that rely solely on placeholders are not fully accessible to screen readers. Additionally, the custom vendor selection buttons lacked the semantic structure of a radio group, making it difficult for assistive technologies to interpret them correctly.
📸 Before/After
(Visuals remain unchanged, improvements are purely semantic/accessibility-focused at the DOM level)
♿ Accessibility
role="radiogroup"andaria-labelledbyto the Vendor selection container.role="radio"andaria-checkedto individual Vendor buttons.aria-label="Repository"andaria-label="Branch"to inputs that previously relied only on placeholders.labelwith itsTextareausinghtmlForandid.PR created automatically by Jules for task 14960006553260014169 started by @parvezk