Skip to content

Style inline code as monospace chips - #480

Open
andrewleesteele wants to merge 3 commits into
mainfrom
hypeship/inline-code-styling
Open

Style inline code as monospace chips#480
andrewleesteele wants to merge 3 commits into
mainfrom
hypeship/inline-code-styling

Conversation

@andrewleesteele

@andrewleesteele andrewleesteele commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Inline code variables 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: Inter config 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-group with their own mono styling.

This adds a scoped :not(pre) > code rule in style.css:

  • monospace font stack
  • subtle background chip + border (respects the site-wide border-radius: 0)
  • normal letter-spacing and 0.875em sizing

Includes 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

  • No page content changed — CSS only.
  • Table inline code keeps its existing dedicated sizing/padding (higher specificity) and picks up the new mono font.

🤖 Generated with Claude Code


Note

Low Risk
Presentation-only CSS with scoped selectors; no content, auth, or data paths touched.

Overview
Adds global style.css rules 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.dark override 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.

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>
@mintlify

mintlify Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated (UTC)
Kernel 🟢 Ready View Preview Aug 5, 2026, 4:00 PM

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

@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Error Error Aug 5, 2026 10:48pm

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>
cursor[bot]
cursor Bot approved these changes Aug 5, 2026
@andrewleesteele
andrewleesteele marked this pull request as ready for review August 5, 2026 22:40

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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 selector

Re-evaluation

  • Prior approval from this automation is on the current HEAD (9e8d605)
  • Diff and risk level are unchanged — approval stands; not re-approving
Open in Web View Automation 

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>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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 (9e8d60532e25d3): 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 to 32e25d3
  • Risk level remains Very Low — no increase, so approval is not revoked and is not re-issued
Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

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.

1 participant