🎨 Palette: Documentation copy links and a11y polish#28
Conversation
- Added "Copy Link" buttons to documentation sections for better shareability. - Implemented multi-modal feedback (icon + aria-label) for copy interactions. - Improved search accessibility by removing shortcut hints from aria-labels. - Fixed build stability by replacing replaceAll with global regex. - Added localized i18n keys for copy link features. - Updated Palette's journal with new UX and a11y learnings. Co-authored-by: MilakyS <155958485+MilakyS@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. |
✅ Deploy Preview for melodious-queijadas-3cc817 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
1 participant
💡 What: The UX enhancement added
aria-labelfor search inputs to exclude keyboard shortcut hints, providing a cleaner experience for screen reader users.String.prototype.replaceAllto global regexreplaceacross all repository scripts to ensure compatibility in Cloudflare Workers and older browser environments.🎯 Why: The user problem it solves
♿ Accessibility: Any a11y improvements made
aria-labelfeedback loop to copy buttons.aria-labelfor screen reader clarity.📸 Before/After
PR created automatically by Jules for task 9571414707835561932 started by @MilakyS