docs: change primary color from blue to indigo#444
Conversation
Align with the cpp-linter organization hub site branding. Indigo (#4051b5) better matches the cpp-linter logo and identity.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
WalkthroughUpdated the MkDocs Material theme primary palette color from blue to indigo for automatic, light, and dark modes, and removed a custom header/drawer background-color rule from ChangesTheme Palette & Styles
🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
So, why not remove this CSS rule? cpp-linter-action/docs/stylesheets/extra.css Lines 5 to 8 in 39f903b IIRC, I made the primary color different from the indigo used in the header/nav because the indigo color made link text hard to read in dark mode: But if that's what you want, then I'm ok with it. |
|
Thanks for pointing that out. I've removed the CSS rule since the palette already handles it now. For the dark mode concern, I checked the rendered result after the change and the indigo links still look readable to me. They're easy to distinguish from the surrounding text, so I think we're OK there. I'd prefer to keep the colors aligned with the cpp-linter hub site for consistency. |
Now that the palette primary color is indigo, the manual CSS override for .md-header and mobile nav drawer is no longer needed.


Summary
Unify the MkDocs primary color with the cpp-linter.github.io hub site.
primary: blue→primary: indigoacross all three palette modes#4051b5(indigo) for mobile nav — now palette matchesSummary by CodeRabbit