Conversation
The banner used var(--vp-c-brand-1) as its background, but VitePress inverts the brand color between themes: in light mode it resolves to #3451b2 (dark indigo) — white text reads fine — but in dark mode it flips to #a8b1ff (light lavender), which makes the white banner text nearly invisible. Hardcode #3451b2 so the banner stays dark-indigo on white text in both themes. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Greptile SummaryFixes banner readability in dark mode by replacing the VitePress theme CSS variable Confidence Score: 5/5This is a safe, minimal CSS-only docs fix with no logic or data concerns. Single-line change in a docs CSS file that correctly addresses the described dark-mode contrast issue. No P0/P1 findings. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[VitePress Theme Mode] --> B{Light or Dark?}
B -->|Light mode| C["--vp-c-brand-1 = #3451b2 (dark indigo)"]
B -->|Dark mode| D["--vp-c-brand-1 = #a8b1ff (light lavender)"]
C --> E["Banner bg: #3451b2 ✅ white text readable"]
D --> F["Banner bg: #a8b1ff ❌ white text invisible (before fix)"]
D --> G["Banner bg: #3451b2 ✅ white text readable (after fix)"]
style F fill:#ff9999
style E fill:#99ff99
style G fill:#99ff99
Reviews (1): Last reviewed commit: "docs: use solid bg for banner so dark-mo..." | Re-trigger Greptile |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #601 +/- ##
=======================================
Coverage 79.03% 79.03%
=======================================
Files 48 48
Lines 7235 7235
Branches 7235 7235
=======================================
Hits 5718 5718
Misses 1140 1140
Partials 377 377 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary
White banner text was unreadable in dark mode because VitePress inverts `--vp-c-brand-1` per theme:
Hardcode the banner bg to `#3451b2` so it stays dark with white text in both themes.
\U0001F916 Generated with Claude Code
Note
Low Risk
Low risk: a single CSS tweak hardcodes the banner background color to avoid theme-variable inversion; no logic or data flow changes.
Overview
For the docs site announcement banner, the background color is now hardcoded to
#3451b2instead of usingvar(--vp-c-brand-1), keeping white banner text readable in dark mode as well as light mode.Reviewed by Cursor Bugbot for commit 9fa2bc1. Bugbot is set up for automated code reviews on this repo. Configure here.