🎨 Palette: [UX improvement] Add context-aware confirmation dialogs and ARIA labels - #81
🎨 Palette: [UX improvement] Add context-aware confirmation dialogs and ARIA labels#81rauhmaru wants to merge 1 commit into
Conversation
- Use `data-name` attribute for context-aware inline confirmation dialogs to prevent JavaScript syntax errors and injection vulnerabilities. - Add descriptive `aria-label`s to the Edit and Remove action buttons across all primary list views (Products, Types, and Cocktails) for improved screen reader accessibility. Co-authored-by: rauhmaru <2289237+rauhmaru@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. |
1 participant
💡 What
Updated the generic generic removal confirmations ("Remover este produto?") to context-aware confirmations (e.g. "Remover Gin Tônica?"). I also added descriptive ARIA labels to the Edit and Remove buttons in the inventory, types, and recipe lists.
🎯 Why
This solves two user problems:
data-nameattribute to prevent Javascript syntax errors or injections if a product name has quotes or special characters.📸 Before/After
Changes are structural and invisible visually; however, they significantly improve the underlying structure and assistive technology support. The confirmation popup will now properly display the name of the item.
♿ Accessibility
Applied templated
aria-labelattributes to action buttons which previously lacked context. Evaluated with Playwright frontend test scripts.PR created automatically by Jules for task 5559698159673994292 started by @rauhmaru