Skip to content

docs: change primary color from blue to indigo#444

Merged
shenxianpeng merged 2 commits into
mainfrom
fix/unify-color-indigo
Jun 9, 2026
Merged

docs: change primary color from blue to indigo#444
shenxianpeng merged 2 commits into
mainfrom
fix/unify-color-indigo

Conversation

@shenxianpeng

@shenxianpeng shenxianpeng commented Jun 8, 2026

Copy link
Copy Markdown
Member

Summary

Unify the MkDocs primary color with the cpp-linter.github.io hub site.

  • Changed primary: blueprimary: indigo across all three palette modes
  • Indigo matches the cpp-linter organization logo and brand identity
  • The extra.css already uses #4051b5 (indigo) for mobile nav — now palette matches

Summary by CodeRabbit

  • Style
    • Updated the documentation theme primary color from blue to indigo across automatic, light, and dark modes for improved visual consistency.
    • Removed a custom header/background color override to align header appearance with the new theme palette.

Align with the cpp-linter organization hub site branding.
Indigo (#4051b5) better matches the cpp-linter logo and identity.
@shenxianpeng shenxianpeng requested a review from a team as a code owner June 8, 2026 16:53
@shenxianpeng shenxianpeng requested review from 2bndy5 and removed request for a team June 8, 2026 16:53
@shenxianpeng shenxianpeng marked this pull request as draft June 8, 2026 16:53
@shenxianpeng shenxianpeng added the documentation Improvements or additions to documentation label Jun 8, 2026
@shenxianpeng shenxianpeng changed the title fix: change primary color from blue to indigo docs: change primary color from blue to indigo Jun 8, 2026
@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 6fca8dc4-7e5c-47b1-90a3-bf38a6a31302

📥 Commits

Reviewing files that changed from the base of the PR and between d4c6afc and 8514a5b.

📒 Files selected for processing (1)
  • docs/stylesheets/extra.css
💤 Files with no reviewable changes (1)
  • docs/stylesheets/extra.css

Walkthrough

Updated 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 docs/stylesheets/extra.css.

Changes

Theme Palette & Styles

Layer / File(s) Summary
Primary color and header background removal
mkdocs.yml, docs/stylesheets/extra.css
Theme primary palette changed from blue to indigo in automatic, light, and dark modes; deleted the .md-header, .md-nav--primary .md-nav__title[for="__drawer"] { background-color: #4051b5; } rule.

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: updating the primary color from blue to indigo in the MkDocs documentation configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/unify-color-indigo

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@shenxianpeng shenxianpeng marked this pull request as ready for review June 8, 2026 17:03
@github-actions github-actions Bot added the bug Something isn't working label Jun 8, 2026
@2bndy5

2bndy5 commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

The extra.css already uses #4051b5 (indigo) for mobile nav — now palette matches

So, why not remove this CSS rule?

.md-header,
.md-nav--primary .md-nav__title[for="__drawer"] {
background-color: #4051b5;
}

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:

on main:
image

after this patch:
image

But if that's what you want, then I'm ok with it.

@shenxianpeng

shenxianpeng commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

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.
@shenxianpeng shenxianpeng removed the bug Something isn't working label Jun 9, 2026
@github-actions github-actions Bot added the bug Something isn't working label Jun 9, 2026
@shenxianpeng shenxianpeng merged commit 6e412de into main Jun 9, 2026
8 checks passed
@shenxianpeng shenxianpeng deleted the fix/unify-color-indigo branch June 9, 2026 06:49
@shenxianpeng shenxianpeng removed the bug Something isn't working label Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants