Skip to content

fix(docs): set Mintlify theme colors to blue#470

Merged
SantiagoDePolonia merged 1 commit into
mainfrom
fix/mintlify-primary-blue
Jul 4, 2026
Merged

fix(docs): set Mintlify theme colors to blue#470
SantiagoDePolonia merged 1 commit into
mainfrom
fix/mintlify-primary-blue

Conversation

@SantiagoDePolonia

@SantiagoDePolonia SantiagoDePolonia commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Set Mintlify docs theme colors in docs.json to the prompt-cache blue family for light and dark appearances.
  • Revert the accidental Discord-only color override from the prior PR.

Validation

  • jq empty docs/docs.json
  • mint validate

Summary by CodeRabbit

  • Style
    • Updated the app’s theme colors to a brighter blue palette.
    • Refreshed the Discord navigation icon colors in both light and dark modes for better visual consistency.

@mintlify

mintlify Bot commented Jul 4, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
gomodel 🟢 Ready View Preview Jul 4, 2026, 2:49 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@coderabbitai

coderabbitai Bot commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Updated theme color values in the documentation configuration and stylesheet: docs.json's primary/light/dark color palette changed, and style.css's Discord navigation icon CSS custom properties updated to matching hex values.

Changes

Theme color update

Layer / File(s) Summary
Color palette update
docs/docs.json, docs/style.css
Primary color changed to #2563eb with new light/dark values in docs.json; corresponding Discord nav CSS variables updated in style.css.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Poem

A hop through hex codes, blue and bright,
Old browns fade into the light,
Discord's icon shifts its hue,
Docs now dressed in azure too,
A rabbit thumps in colorful delight! 🐰🎨

🚥 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 clearly summarizes the main change to the docs theme colors and is concise and specific.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/mintlify-primary-blue

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/style.css`:
- Around line 9-12: The Discord navigation icon styling is still using hardcoded
brown/green values instead of the docs theme palette. Update the CSS rule for
the Discord nav anchor in docs/style.css to use the blue palette defined by
docs/docs.json, and keep the change scoped to the existing --discord-nav-light
and --discord-nav-dark variables so the nav item matches the rest of the UI.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: f56355c6-9e22-4d5e-b9f6-01f0c380a3ab

📥 Commits

Reviewing files that changed from the base of the PR and between b289bf1 and b344e72.

📒 Files selected for processing (2)
  • docs/docs.json
  • docs/style.css

Comment thread docs/style.css
Comment on lines 9 to 12
#navigation-items .nav-anchor[href="https://discord.gg/gaEB9BQSPH"] {
--discord-nav-light: #2563eb;
--discord-nav-dark: #60a5fa;
--discord-nav-light: #755c3d;
--discord-nav-dark: #4ade80;
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n== docs/style.css ==\n'
nl -ba docs/style.css | sed -n '1,40p'

printf '\n== docs/docs.json (palette-related lines) ==\n'
rg -n '"(primary|discord|theme|color|palette|blue)"' docs/docs.json -n -C 2 || true

printf '\n== Discord nav variable references ==\n'
rg -n 'discord-nav-(light|dark)|discord\.gg/gaEB9BQSPH|nav-anchor' docs -n -C 2 || true

Repository: ENTERPILOT/GoModel

Length of output: 218


🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n== docs/style.css ==\n'
sed -n '1,40p' docs/style.css | cat -n

printf '\n== docs/docs.json (palette-related lines) ==\n'
rg -n '"(primary|discord|theme|color|palette|blue)"' docs/docs.json -C 2 || true

printf '\n== Discord nav variable references ==\n'
rg -n 'discord-nav-(light|dark)|discord\.gg/gaEB9BQSPH|nav-anchor' docs -C 2 || true

Repository: ENTERPILOT/GoModel

Length of output: 4683


🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n== docs/style.css ==\n'
sed -n '1,40p' docs/style.css | cat -n

printf '\n== docs/docs.json ==\n'
sed -n '1,220p' docs/docs.json | cat -n

printf '\n== Discord nav variable references ==\n'
rg -n 'discord-nav-(light|dark)|discord\.gg/gaEB9BQSPH|nav-anchor' docs -C 2 || true

Repository: ENTERPILOT/GoModel

Length of output: 10542


Use the theme palette for the Discord nav icon. docs/style.css:9-12 still hardcodes --discord-nav-light to #755c3d and --discord-nav-dark to #4ade80, but docs/docs.json defines the docs palette as #2563eb / #60a5fa / #1d4ed8. Swap these to the blue theme colors so the Discord item matches the rest of the UI.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/style.css` around lines 9 - 12, The Discord navigation icon styling is
still using hardcoded brown/green values instead of the docs theme palette.
Update the CSS rule for the Discord nav anchor in docs/style.css to use the blue
palette defined by docs/docs.json, and keep the change scoped to the existing
--discord-nav-light and --discord-nav-dark variables so the nav item matches the
rest of the UI.

@greptile-apps

greptile-apps Bot commented Jul 4, 2026

Copy link
Copy Markdown

Confidence Score: 5/5

Safe to merge with minimal risk.

The PR only changes Mintlify theme colors and CSS variables for the docs navigation. No runtime application code, data handling, or security-sensitive paths are affected.

No files require special attention.

T-Rex T-Rex Logs

What T-Rex did

  • Ran jq to validate that the docs.json file is empty and it completed with exit code 0.
  • Mintlify reported a successful build validation and exited with code 0.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (1): Last reviewed commit: "fix(docs): set Mintlify theme colors to ..." | Re-trigger Greptile

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@SantiagoDePolonia SantiagoDePolonia merged commit 832c0ed into main Jul 4, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants