Skip to content

update fonts for better readability, removed satoshi font#419

Merged
sriramveeraghanta merged 1 commit intomasterfrom
fix-fonts
Mar 24, 2026
Merged

update fonts for better readability, removed satoshi font#419
sriramveeraghanta merged 1 commit intomasterfrom
fix-fonts

Conversation

@vihar
Copy link
Contributor

@vihar vihar commented Mar 24, 2026

Description

Replaced Satoshi font (4 static weight files, ~228KB) with Inter Variable (single woff2, ~344KB, weights 100–900) to match the developer docs site
Added font rendering optimizations (antialiased, grayscale, optimizeLegibility) and Inter OpenType features (cv01, cv02, zero)
Added typography tuning for .vp-doc body and headings (line-height, letter-spacing)
Fixed .api-endpoint-badge .method and .path to use var(--vp-font-family-mono) instead of hardcoded monospace
Updated font preload in config from Satoshi to Inter Variable

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Improvement (change that would cause existing functionality to not work as expected)
  • Code refactoring
  • Performance improvements
  • Documentation update

Summary by CodeRabbit

  • Style
    • Updated documentation fonts from Satoshi to Inter variable font for improved typography
    • Enhanced text rendering with refined line-height and letter-spacing across headings and content
    • Improved overall legibility and font consistency throughout the documentation

@vercel
Copy link

vercel bot commented Mar 24, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 24, 2026 8:28pm

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Mar 24, 2026

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 75f9beab-b593-4d9b-9297-a637ad21e610

📥 Commits

Reviewing files that changed from the base of the PR and between 42285df and 2d829f2.

⛔ Files ignored due to path filters (9)
  • docs/public/fonts/Inter/InterVariable.woff2 is excluded by !**/*.woff2
  • docs/public/fonts/Satoshi/Satoshi-Bold.woff is excluded by !**/*.woff
  • docs/public/fonts/Satoshi/Satoshi-Bold.woff2 is excluded by !**/*.woff2
  • docs/public/fonts/Satoshi/Satoshi-Light.woff is excluded by !**/*.woff
  • docs/public/fonts/Satoshi/Satoshi-Light.woff2 is excluded by !**/*.woff2
  • docs/public/fonts/Satoshi/Satoshi-Medium.woff is excluded by !**/*.woff
  • docs/public/fonts/Satoshi/Satoshi-Medium.woff2 is excluded by !**/*.woff2
  • docs/public/fonts/Satoshi/Satoshi-Regular.woff is excluded by !**/*.woff
  • docs/public/fonts/Satoshi/Satoshi-Regular.woff2 is excluded by !**/*.woff2
📒 Files selected for processing (2)
  • docs/.vitepress/config.ts
  • docs/.vitepress/theme/style.css

📝 Walkthrough

Walkthrough

The documentation site font system was migrated from Satoshi to Inter. The preload configuration was updated to reference a single Inter variable font file instead of multiple Satoshi font files. CSS font-face declarations were replaced with a single variable font definition, and typography properties were adjusted globally.

Changes

Cohort / File(s) Summary
Font Preload Configuration
docs/.vitepress/config.ts
Replaced preloading of two Satoshi font files with a single Inter variable font file in VitePress head configuration.
Font System & Typography
docs/.vitepress/theme/style.css
Migrated from multiple Satoshi @font-face declarations to a single Inter variable font; updated --vp-font-family-base CSS variable; added global body and heading typography adjustments (line-height, letter-spacing); converted hardcoded mono font references to use var(--vp-font-family-mono) variable.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 The fonts have changed, a fresh new face,
From Satoshi's charm to Inter's grace,
Variable weights in one smooth file,
The docs now render with modern style! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: replacing the Satoshi font with Inter for improved readability, which is the primary modification across both config and CSS files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-fonts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@vihar vihar requested a review from sriramveeraghanta March 24, 2026 20:29
@sriramveeraghanta sriramveeraghanta merged commit 5ce1e24 into master Mar 24, 2026
4 of 5 checks passed
@sriramveeraghanta sriramveeraghanta deleted the fix-fonts branch March 24, 2026 20:30
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