Skip to content

update fonts for better readability, removed satoshi font#249

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

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

Conversation

@vihar
Copy link
Contributor

@vihar vihar commented Mar 24, 2026

Description

Font swap: Replaced Satoshi (4 static weight files, ~228KB) with Inter Variable (single woff2 file, ~344KB covering weights 100-900).

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
    • Upgraded typography with a modern variable font for improved rendering quality.
    • Enhanced text rendering, smoothing, and readability across documentation.
    • Optimized font loading performance through preloading.

@coderabbitai
Copy link

coderabbitai bot commented Mar 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0d078d60-9d86-43db-a209-fb7cac9dc6b5

📥 Commits

Reviewing files that changed from the base of the PR and between cb28431 and 5254a89.

⛔ 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.mts
  • docs/.vitepress/theme/style.css

📝 Walkthrough

Walkthrough

This pull request migrates the documentation's typography from the Satoshi font to Inter variable font, adding font preloading optimization and enhancing typography rendering rules across the documentation site.

Changes

Cohort / File(s) Summary
Font Migration to Inter
docs/.vitepress/config.mts, docs/.vitepress/theme/style.css
Replaced Satoshi font-face definitions with a single Inter variable font file (/fonts/Inter/InterVariable.woff2). Added preload link tag in config for performance. Updated base font stack variable to Inter. Enhanced global typography with font smoothing, improved text rendering, and tuned spacing for headings.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Satoshi bids farewell with grace and style,
But Inter arrives, so versatile and spry,
Variable fonts now dance and flex,
From 100 to 900 weights perplex,
One file replaces many—oh, what a compile! ✨📝

🚥 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 summarizes the main change: replacing Satoshi font with Inter variable font and tuning CSS for improved readability.
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.

@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)
developer-docs Ready Ready Preview, Comment Mar 24, 2026 8:14pm

Request Review

@sriramveeraghanta sriramveeraghanta merged commit abbea9f into master Mar 24, 2026
8 checks passed
@sriramveeraghanta sriramveeraghanta deleted the fix-fonts branch March 24, 2026 20:22
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