🎨 Palette: Standardize search inputs in LyShine Animation UI

[Zee2401]

💡 What: Added placeholderText, clearButtonEnabled, and accessibleName to search/filter fields in the LyShine Animation UI. Established a buddy link between the filter label and input in the Find dialog.
🎯 Why: Improving search input UX by providing clear intent (placeholder), a quick way to reset (clear button), and better accessibility (buddy link and accessible names).
♿ Accessibility:

• Linked "Enter filter:" label to its input field using the buddy property.
• Added accessibleName "Search" to search fields for screen reader support.

---

PR created automatically by Jules for task 7509735781729463753 started by @Zee2401

Summary by CodeRabbit

• New Features
  • Search and filter input fields now display placeholder text for improved clarity.
  • Added clear button functionality to search and filter inputs for quick field reset.
  • Enhanced accessibility for search input fields throughout the application.

[google-labs-jules]

👋 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 @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: af5f47a4-9d41-4a62-a3ee-f2471dd8e5e2

📥 Commits

Reviewing files that changed from the base of the PR and between 876034b and 9a898d1.

📒 Files selected for processing (3)

• .Jules/palette.md
• Gems/LyShine/Code/Editor/Animation/UiAnimViewFindDlg.ui
• Gems/LyShine/Code/Editor/Animation/UiAnimViewNodes.ui

---

📝 Walkthrough

Walkthrough

This PR standardizes search and filter input patterns across Qt editor UIs. The .Jules/palette.md guidance document is updated to specify the pattern, then the pattern is applied consistently to two animation editor UI files with placeholder text, clear buttons, accessible names, and label buddy relationships.

Changes

Search/Filter Input Standardization

Layer / File(s)	Summary
Pattern documentation and guidance .Jules/palette.md	Updated palette guidance documents the standard Qt search/filter QLineEdit pattern with placeholder text, clear button, accessible name, and label buddy linkage.
UiAnimViewFindDlg pattern implementation Gems/LyShine/Code/Editor/Animation/UiAnimViewFindDlg.ui	The "Enter filter" QLabel adds a buddy relationship to the FILTER input. The FILTER QLineEdit gains placeholder text ("Search..."), an enabled clear button, and an accessible name ("Search").
UiAnimViewNodes pattern implementation Gems/LyShine/Code/Editor/Animation/UiAnimViewNodes.ui	The searchField QLineEdit gains placeholder text ("Search…"), an enabled clear button, and an accessible name ("Search").

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• Zee2401/o3de#1: Both PRs update the shared .Jules/palette.md guidance for the standard search/filter QLineEdit UX and apply the same placeholder/clear (and linked accessibility/buddy) pattern in corresponding editor search/filter UIs.

Poem

🐰 A filter so clear, with placeholder so bright,
Clear buttons that shine, oh what a delight!
Buddy labels now link, accessibility's right,
Standardized searches—the UI's just right! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	The description covers what was changed and why, though it lacks explicit testing information required by the template.	Add a 'How was this PR tested?' section describing any testing performed, such as UI verification or accessibility testing with screen readers.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: standardizing search input properties across LyShine Animation UI files, matching the actual modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch palette/standardize-lyshine-search-7509735781729463753

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}