docs: replace Quantify with Montserrat 800 in navbar title#20997
docs: replace Quantify with Montserrat 800 in navbar title#20997bloxster wants to merge 7 commits intorelease/3.4from
Conversation
yperbasis
left a comment
There was a problem hiding this comment.
- Footer still references Quantify — will silently fall back after this PR (bug)
docs/site/src/theme/Footer/index.tsx:47:
<span style={{fontFamily: "'Quantify', sans-serif", fontWeight: 700, ...}}>erigon.tech
With the @font-face removed and Quantify.woff2 deleted, this brand text will fall back to whatever the browser resolves sans-serif to — different from both Quantify and the navbar's
new Montserrat 800. The PR description says "No change to headings, body, or any other element", but the footer brand text will visibly change.
The PR reasoning explicitly calls out that "Quantify is the brand font on erigon.tech (logo wordmark only, two occurrences: header + footer)" — so the same fix that's being applied to
the navbar should be applied to the footer in the same PR. Otherwise the navbar and footer brand text will diverge.
- docs/site/DESIGN-SPEC.md is now stale
Four references contradict the new state:
- Line 119: "Font: Quantify, weight 700, letter-spacing 0.04em" (navbar title)
- Line 159: "Brand text: Quantify font, weight 700" (footer brand)
- Line 272: @import url('https://fonts.cdnfonts.com/css/quantify'); (already stale from #20991, but worth fixing here)
- Line 280: | Navbar title, footer brand | Quantify | 700 |
The doc was written as a brand spec, so leaving it describing a font that no longer ships is confusing for future contributors.
Addresses review feedback on #20997: - Footer/index.tsx: replace inline Quantify/700 with Montserrat/800 to match the navbar change; without this the footer brand text would silently fall back to sans-serif after the @font-face and .woff2 are removed - DESIGN-SPEC.md: update four stale Quantify references (navbar title, footer brand, CDN @import, font-roles table) to reflect Montserrat 800 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
There was a problem hiding this comment.
Pull request overview
Updates the docs site branding typography by removing the custom Quantify font usage and switching the navbar title (and footer brand text) to the already self-hosted Montserrat ExtraBold (800), aligning the docs UI with the stated brand guidelines.
Changes:
- Removed the
@font-facefor Quantify and updated.navbar__titletoMontserratweight800. - Updated the footer “erigon.tech” brand text to use
Montserratweight800. - Adjusted the design spec to reflect the new navbar/footer brand font choice (but see documentation note below).
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/site/src/theme/Footer/index.tsx | Switch footer brand text font from Quantify to Montserrat 800. |
| docs/site/src/css/custom.css | Remove Quantify font-face and update navbar title typography to Montserrat 800. |
| docs/site/DESIGN-SPEC.md | Update spec to state Montserrat 800 for navbar title/footer brand. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…font-face Agent-Logs-Url: https://github.com/erigontech/erigon/sessions/80b1bb4b-9da9-4c6b-b0a3-18baed62cf6d Co-authored-by: bloxster <40316187+bloxster@users.noreply.github.com>
yperbasis
left a comment
There was a problem hiding this comment.
Hmm. We can keep Quantify because we have the licence now, right?
|
Yes, correct, we are licensed. |
…tion Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Closing because we can keep Quantify. |
|
@yperbasis the PR about fixing CSS with the right place for each font, please approve / merge. |
Typography (custom.css): - Negative letter-spacing on h1 (-0.015em) and h2 (-0.01em) — Montserrat ExtraBold reads tighter at display sizes - Treat h6 as a section label (uppercase, 0.85rem, muted) — gives real visual hierarchy below h1–h5 instead of yet-another-bold-heading - Heading line-height 1.2 (was Docusaurus default 1.25), font-smoothing + optimizeLegibility on <html> - Vertical rhythm: 2.5rem above .markdown h2, 1.75rem above h3 - scroll-margin-top on all .markdown headings = navbar + 1rem so anchor links land just below the sticky navbar Installation scroll fix (get-started/installation/index.mdx): The OS-method links no longer scroll to the wrong place on the first click. Two issues addressed: 1. Section anchor mapping — clicking an install method now scrolls to the parent section heading (`#linuxmacos`, `#windows`, `#all-operating-systems`) rather than the <details> element itself, and the URL hash is updated to match via history.replaceState. 2. Layout-stability sequencing — the previous code tried to scroll while close/open animations were still in flight, which caused non-deterministic landing positions (especially on second clicks that crossed sections, e.g. macOS→Windows). The new flow: close other open methods → open the requested one → poll document.body.scrollHeight via requestAnimationFrame until stable for 6 frames (~100ms) → smooth-scroll to the section heading. Also closes any other open install method first so only one is visible at a time. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
@bloxster Please update the title and description then to match the end result. |
Summary
Originally a typography fix for the brand font; expanded into broader design tuning on the same scope.
1. Quantify → Montserrat 800 (the original fix)
@font-facedeclaration for Quantify fromcustom.cssdocs/site/static/fonts/Quantify.woff2(non-commercial license, wrong usage scope).navbar__titleand footer brand to use Montserrat weight 800 (already self-hosted) instead of Quantify weight 700Reason: Quantify is the brand font on erigon.tech for the logo wordmark only (header + footer wordmark). Per brand guidelines, everything else must use Montserrat / Nunito Sans.
2. Typography polish (
custom.css)Refinements that target Montserrat ExtraBold at display sizes — no new fonts, no size changes:
h1(-0.015em) andh2(-0.01em)h6treated as a section label (uppercase, 0.85rem, muted) — gives real visual hierarchy--ifm-heading-line-height: 1.2(was Docusaurus default 1.25)text-rendering: optimizeLegibilityon<html>.markdown h2/h3scroll-margin-topon all.markdownheadings so in-page anchor links land just below the sticky navbar3. Installation page scroll behavior fix (
get-started/installation/index.mdx)Restores and improves the smooth scroll + auto-open behavior on the install-method tabs:
#linuxmacos,#windows,#all-operating-systems) — not the<details>element — and updates the URL hash.document.body.scrollHeightperrequestAnimationFrameuntil stable for 6 frames (~100ms), then scrolls — so the smooth-scroll target is always at its final position.Visual change
Test plan
pnpm run start— local dev, all scroll/open paths verified across Brave, Chromium, Firefox, LibreWolfcd docs/site && pnpm run build— clean build/fundamentals/logs,/interacting-with-erigon/trace)🤖 Generated with Claude Code