🎨 Palette: [UX improvement] - Profile search accessibility and filter bug fix - #379
🎨 Palette: [UX improvement] - Profile search accessibility and filter bug fix#379yeboster wants to merge 1 commit into
Conversation
…reset * Conditionally render trailing clear icon in Textfield * Use 'clear search' aria-label instead of generic 'cancel' * Dynamically bind withTrailingIcon property to preserve layout * Add missing 'else' statement to reset filtered domains when search is empty * Add .jules/ and static/ to .prettierignore Co-authored-by: yeboster <23556525+yeboster@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
👋 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:
Textfieldso it only appears when there's text to clear..jules/andstatic/to.prettierignoreto prevent auto-formatting failures on generated files.🎯 Why:
pnpm run formatcommand was failing because it attempted to format newly generated.jules/palette.mdand compiledstatic/fonts.📸 Before/After: Visual change is minor (clear icon disappears when input is empty).
♿ Accessibility:
aria-label="cancel"with a more contextually accuratearia-label="clear search"for screen reader users.PR created automatically by Jules for task 16019892818908916410 started by @yeboster