π¨ Palette: [UX improvement] Add empty state for results list - #496
π¨ Palette: [UX improvement] Add empty state for results list#496xbmc4lyfe wants to merge 1 commit into
Conversation
Added a centered empty state label to the results list. Kodi custom lists do not automatically handle empty states, leading to a confusing blank screen when there are no items. Before, a blank screen was shown. After, "No results found" is displayed when the list is empty. This provides clear textual feedback to users when no items are available, improving usability and accessibility. Co-authored-by: xbmc4lyfe <273732874+xbmc4lyfe@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. |
Dependency Reviewβ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
|
No actionable comments were generated in the recent review. π βΉοΈ Recent review infoβοΈ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: π Files selected for processing (1)
π Recent review detailsβ° Context from checks skipped due to timeout. (7)
π Additional comments (1)
π WalkthroughSummary by CodeRabbit
WalkthroughThe results dialog now shows a centered βNo results foundβ label when results list 50 contains zero items. ChangesResults empty state
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: Poem
π₯ Pre-merge checks | β 5β Passed checks (5 passed)
β¨ Finishing Touchesπ§ͺ Generate unit tests (beta)
β¨ Simplify code
Warning Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption. 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. Comment |
Up to standards βπ’ Issues
|
π‘ What
Added a centered "No results found" empty state label to the results list in
results-dialog.xml.π― Why
Kodi custom lists (
<control type="list">) do not automatically handle empty states. If a search or filter returned zero results, the dialog would previously display a confusing blank screen without any feedback, leaving the user to wonder if it was still loading or broken.πΈ Before/After
Before: Blank screen when
Container(50).NumItemsis 0.After: A centered, muted "No results found" message is displayed clearly.
βΏ Accessibility
Provides clear textual feedback to all users when no items are available, significantly improving usability and reducing cognitive load.
PR created automatically by Jules for task 18249113213817315233 started by @xbmc4lyfe