Style inline code as monospace chips - #480
Open
andrewleesteele wants to merge 3 commits into
Open
Conversation
The global Inter font family was applying to bare inline <code> spans, rendering variables in a proportional font with no visual treatment so they read as malformed prose. Force a monospace stack, subtle background, and normal spacing on inline code (code blocks inside <pre> unaffected). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Grey chip read as sterile. Use kernel green (#81b300) as a low-alpha fill with a green border, keeping the existing charcoal/light text per the brand rule (green as fill, never green body text). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
andrewleesteele
marked this pull request as ready for review
August 5, 2026 22:40
There was a problem hiding this comment.
Stale comment
Risk assessment: Very Low
Verdict: Already approved — no re-approval needed
Evidence (from diff only)
- 1 file changed:
style.css(+23 / −0)- Presentational CSS only: monospace stack, sizing, padding, and green-tinted chip styles for
:not(pre) > code, plus a dark-mode variant- No markdown/content, JS/TS, config, infra, auth, or schema changes
- No CODEOWNERS file in the repo
Why Very Low
- Small, cosmetic styling change on a documentation site
- No production application logic or shared backend systems
- Blast radius is visual only (inline code appearance);
<pre>code blocks are excluded by the selectorRe-evaluation
- Prior approval from this automation is on the current HEAD (
9e8d605)- Diff and risk level are unchanged — approval stands; not re-approving
Sent by Cursor Automation: Assign PR reviewers
Switch the inline-code chip fill from kernel green to a light gold (#cab168, the docs accent), keeping charcoal/off-white text. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
Risk assessment: Very Low
Verdict: Already approved — risk unchanged after synchronize; approval stands (not re-approving)
Evidence (from diff only)
- 1 file changed:
style.css(+22 / −0 vs base) - Presentational CSS only: monospace stack, sizing, padding, and gold-tinted chip styles for
:not(pre) > code, plus a dark-mode variant - Latest commit vs prior approved HEAD (
9e8d605→32e25d3): color-token swap only (green → light gold rgba values / comment text) - No markdown/content, JS/TS, config, infra, auth, or schema changes
- No CODEOWNERS file in the repo
Why Very Low
- Small, cosmetic styling change on a documentation site
- No production application logic or shared backend systems
- Blast radius is visual only (inline code appearance);
<pre>code blocks are excluded by the selector
Re-evaluation
- Prior approval from this automation was on
9e8d605; PR synchronized to32e25d3 - Risk level remains Very Low — no increase, so approval is not revoked and is not re-issued
Sent by Cursor Automation: Assign PR reviewers
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Summary
Inline
codevariables in the docs were rendering in the global Inter (proportional) font with no visual treatment — weird spacing, wrong size, and no code-like appearance, so they read as malformed prose instead of code.Root cause: the site's
fonts.family: Interconfig applies broadly, and bare inline<code>spans (e.g.payload,runtimeContext,App) had no dedicated styling to override it. Code blocks were unaffected since they live inside<pre>/.code-groupwith their own mono styling.This adds a scoped
:not(pre) > coderule instyle.css:border-radius: 0)0.875emsizingIncludes a dark-mode variant using the same neutral tokens already used elsewhere in the file.
Test page
Set up as a test on the Developing page, which is dense with inline variables. If it looks good, the same treatment covers inline code across all docs pages (the rule is global). Preview once the Mintlify branch build is up.
Notes
🤖 Generated with Claude Code
Note
Low Risk
Presentation-only CSS with scoped selectors; no content, auth, or data paths touched.
Overview
Adds global
style.cssrules for inline<code>(via:not(pre) > code) so bare variables pick up a monospace stack, subtle gold-tinted chip background/border, and normalized sizing/letter-spacing instead of inheriting Inter from the site font config.<pre>blocks are unchanged.A
html.darkoverride bumps background and border opacity for the same inline treatment in dark mode.Reviewed by Cursor Bugbot for commit 32e25d3. Bugbot is set up for automated code reviews on this repo. Configure here.