Skip to content

docs(nav): remove the Grid mark from the header logo - #911

Merged
patcapulong merged 2 commits into
mainfrom
pat/remove-grid-logo-header
Sep 8, 2026
Merged

docs(nav): remove the Grid mark from the header logo#911
patcapulong merged 2 commits into
mainfrom
pat/remove-grid-logo-header

Conversation

@patcapulong

@patcapulong patcapulong commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Why

The header should read as just the lightspark wordmark, without the Grid mark next to it. The header logo is a single SVG per theme (mintlify/logo/light.svg, dark.svg) with the wordmark, a / divider, and the Grid mark + wordmark baked in, and there was no lightspark-only asset in the repo.

While checking the result locally, the navbar links (GitHub / Book a live demo / Dashboard) had ~48px between them instead of the intended ~16px. That's fixed here too as a separate commit.

Changes

Header logo (ba0bcd0b)

  • Drop the divider and the six Grid paths from both logo SVGs.
  • Trim the canvas from 206×26 to 120×26 so there's no trailing whitespace next to the wordmark.
  • Removed the / divider along with the mark since a lone lightspark / would look dangling.

Navbar link spacing (6dcec875)

  • The Mintlify client now ships Tailwind v4, where space-x-* sets margin-inline-end on every child except the last (v3 set margin-left on every child except the first). Our existing margin-left: 0 reset stopped reaching the space-x-6 on the link list, so the 24px margin stacked on top of the intended 8px gap. Added a direct margin: 0 on the list items rather than broadening the existing wide navbar reset.
  • Mintlify renders <li class="navbar-link"><a>, and our link rules matched both .navbar-link and the anchor, so each item was padded twice and the hover opacity applied twice (~0.5 instead of 0.7). Scoped those rules to the anchor only.

Side effect: auto-generated OG images

Pages without a hard-coded og:image get Mintlify's auto-generated social card, which overlays the docs.json logo on thumbnails.background. Those cards (~39 pages, e.g. /changelog) now show only the lightspark wordmark. This is intended. The 96 pages with a static og:image under mintlify/images/og/ are unchanged, since that branding is baked into the PNGs.

Verification

  • Rendered both SVGs with rsvg-convert; wordmark is intact, no trailing space.
  • Ran mint dev locally from this branch and confirmed the new logo and CSS are served; style.css on main and the current feature branches have identical navbar rules, so this isn't branch-specific.
  • Compared the /changelog OG card from the Mintlify preview against prod to confirm the logo change flows through.

patcapulong and others added 2 commits September 8, 2026 12:48
Trim the divider and the Grid mark + wordmark out of the light and dark
header logos so only the lightspark wordmark shows. Shrinks the SVG
canvas from 206x26 to 120x26 to match the remaining paths.

Co-authored-by: Cursor <cursoragent@cursor.com>
The Mintlify client now ships Tailwind v4, where space-x-* sets
margin-inline-end on every child except the last instead of margin-left
on every child except the first. Our margin-left reset stopped reaching
the space-x-6 on the link list, so the 24px margin stacked on top of the
intended 8px gap. Zero the list items' margin directly.

Also stop matching .navbar-link (the li) in the link rules: Mintlify
renders <li class="navbar-link"><a>, so the li was padded on top of the
anchor and the hover opacity was applied twice.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Sep 8, 2026

Copy link
Copy Markdown

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

2 Skipped Deployments
Project Deployment Actions Updated
grid-flow-builder Ignored Ignored Sep 8, 2026 8:28pm UTC
grid-wallet-demo Ignored Ignored Sep 8, 2026 8:28pm UTC

Request Review

@mintlify

mintlify Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated
Grid 🟢 Ready View Preview Sep 8, 2026, 8:30 PM

@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge with no actionable correctness, security, or repository-rule issues identified.

Summary

  • Removes the Grid divider, mark, and wordmark from both theme-specific header logos.
  • Resizes each SVG canvas to the retained Lightspark wordmark.
  • Resets Tailwind-generated list-item margins and scopes interactive navbar styles to anchors to avoid compounded spacing and opacity.

@patcapulong
patcapulong merged commit 01f0b24 into main Sep 8, 2026
9 checks passed
@patcapulong
patcapulong deleted the pat/remove-grid-logo-header branch September 8, 2026 22:12
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