enh(frontend): use light background for infra icons#3090
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe vendor icon component’s dark-theme background now uses the fixed color ChangesVendor icon theme styling
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
How to categorize this PR?
/area usability
/kind enhancement
What this PR does / why we need it:
Adds a light background color to the infra icons so that they are easily visible in dark mode.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:
Screenshots:
Before:
After:
Summary by CodeRabbit