Skip to content

docs: replace Quantify with Montserrat 800 in navbar title#20997

Open
bloxster wants to merge 7 commits intorelease/3.4from
docs/remove-quantify-font
Open

docs: replace Quantify with Montserrat 800 in navbar title#20997
bloxster wants to merge 7 commits intorelease/3.4from
docs/remove-quantify-font

Conversation

@bloxster
Copy link
Copy Markdown
Collaborator

@bloxster bloxster commented May 5, 2026

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)

  • Removes @font-face declaration for Quantify from custom.css
  • Deletes docs/site/static/fonts/Quantify.woff2 (non-commercial license, wrong usage scope)
  • Updates .navbar__title and footer brand to use Montserrat weight 800 (already self-hosted) instead of Quantify weight 700

Reason: 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:

  • Negative letter-spacing on h1 (-0.015em) and h2 (-0.01em)
  • h6 treated as a section label (uppercase, 0.85rem, muted) — gives real visual hierarchy
  • --ifm-heading-line-height: 1.2 (was Docusaurus default 1.25)
  • Font-smoothing + text-rendering: optimizeLegibility on <html>
  • Vertical rhythm: more breathing room above .markdown h2 / h3
  • scroll-margin-top on all .markdown headings so in-page anchor links land just below the sticky navbar

3. Installation page scroll behavior fix (get-started/installation/index.mdx)

Restores and improves the smooth scroll + auto-open behavior on the install-method tabs:

  • Clicking a method (e.g. Pre-built Binaries) now scrolls to the parent OS section heading (#linuxmacos, #windows, #all-operating-systems) — not the <details> element — and updates the URL hash.
  • Other open install methods are auto-closed so only one is visible at a time.
  • The previous logic raced the close/open animations against the smooth scroll, causing non-deterministic landing on second-clicks across sections. New flow polls document.body.scrollHeight per requestAnimationFrame until stable for 6 frames (~100ms), then scrolls — so the smooth-scroll target is always at its final position.

Visual change

Where Change
Navbar title Quantify → Montserrat ExtraBold
Footer brand Quantify → Montserrat ExtraBold
All h1–h6 Tighter heading line-height, antialiased rendering
h1 / h2 Slight negative tracking
h6 Now a small uppercase label
Install page tabs Smooth scroll + open behavior fully working again

Test plan

  • pnpm run start — local dev, all scroll/open paths verified across Brave, Chromium, Firefox, LibreWolf
  • cd docs/site && pnpm run build — clean build
  • Verify navbar title renders in Montserrat in light + dark mode
  • Confirm no requests to any font CDN in Network tab
  • Sanity-check headings on representative content pages (e.g. /fundamentals/logs, /interacting-with-erigon/trace)

🤖 Generated with Claude Code

Copy link
Copy Markdown
Member

@yperbasis yperbasis left a comment

Choose a reason for hiding this comment

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

  1. 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.

  1. 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>
@bloxster
Copy link
Copy Markdown
Collaborator Author

bloxster commented May 5, 2026

  • Footer: 'Quantify', 700 → 'Montserrat', 800 — navbar and footer now use the same font/weight
  • DESIGN-SPEC.md: all four stale Quantify references updated (navbar title, footer brand, CDN @import line removed, font roles table)

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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-face for Quantify and updated .navbar__title to Montserrat weight 800.
  • Updated the footer “erigon.tech” brand text to use Montserrat weight 800.
  • 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.

Comment thread docs/site/DESIGN-SPEC.md Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.

@bloxster bloxster requested a review from yperbasis May 6, 2026 12:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.

Copy link
Copy Markdown
Member

@yperbasis yperbasis left a comment

Choose a reason for hiding this comment

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

Hmm. We can keep Quantify because we have the licence now, right?

@bloxster
Copy link
Copy Markdown
Collaborator Author

bloxster commented May 7, 2026

Yes, correct, we are licensed.

@bloxster bloxster requested a review from yperbasis May 7, 2026 16:36
…tion

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yperbasis yperbasis closed this May 8, 2026
@yperbasis
Copy link
Copy Markdown
Member

Closing because we can keep Quantify.

@bloxster bloxster reopened this May 8, 2026
@bloxster
Copy link
Copy Markdown
Collaborator Author

bloxster commented May 8, 2026

@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>
@yperbasis
Copy link
Copy Markdown
Member

@bloxster Please update the title and description then to match the end result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants