Fix unreleased dark mode bugs in 4.85.0#45200
Conversation
- #45077: Make laptop-mac.png laptop background transparent (was a solid white rectangle bleeding through the dashboard welcome card in dark mode). - #45094: Use the standard neutral hover/focus border on the AutocompleteDropdown in the Add users modal, matching the rest of Fleet's dropdowns. - #45116 / #45119: Set the DropdownWrapper search input text color so typed text is readable in dark mode (covers the schema Table dropdown on Reports/Policies/Labels and the Transfer host modal).
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
There was a problem hiding this comment.
Pull request overview
This PR addresses several UI dark-mode styling issues in the Fleet frontend by aligning dropdown hover/focus and react-select input text styling with the app’s themed color system.
Changes:
- Updated
AutocompleteDropdownhover/focus border color to use the neutral themed gray ($ui-fleet-black-50) instead of the accent blue. - Updated
DropdownWrapperreact-select custom styles so typed input text uses a themed color (COLORS["core-fleet-black"]), improving contrast in dark mode.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| frontend/pages/admin/TeamManagementPage/TeamDetailsWrapper/UsersPage/components/AutocompleteDropdown/_styles.scss | Aligns hover/focus border color with standard dropdown styling for consistent appearance across themes. |
| frontend/components/forms/fields/DropdownWrapper/DropdownWrapper.tsx | Ensures searchable dropdown input text uses themed color tokens so it remains readable in dark mode. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
WalkthroughThis PR applies styling updates to dropdown components. The Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #45200 +/- ##
==========================================
- Coverage 66.81% 66.81% -0.01%
==========================================
Files 2721 2721
Lines 218947 218947
Branches 10627 10627
==========================================
- Hits 146282 146281 -1
- Misses 59514 59515 +1
Partials 13151 13151
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary
Four unreleased dark mode bugs from the #g-first-impressions board.
laptop-mac.pngtransparent so the dashboard "Welcome to Fleet" card doesn't show a white rectangle behind the laptop in dark mode. Two-pass cleanup: flood-fill from the edges removes the solid white background; edge pixels adjacent to transparent regions get alpha proportional to their distance from white so anti-aliased edges fade instead of leaving a halo.AutocompleteDropdown(Add users modal) now uses$ui-fleet-black-50for hover/focus border, matching Fleet's standardDropdowncomponent. Previously rendered as$core-vibrant-blue, which reads as blue in light mode and violet in dark mode.DropdownWrapper's searchable input was relying on react-select's default text color (hsl(0, 0%, 20%)), unreadable on the dark menu background. Settingcolor: COLORS["core-fleet-black"]lets the CSS custom property resolve to light text in dark mode. Fixes both the schema Table dropdown (Reports / Policies / Labels) and the Transfer host modal.Note: #45117 ("Various" role label) turned out to be intentional; not included.
Test plan
Summary by CodeRabbit