You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
fix(privacy): self-host the Manrope font instead of loading it from Google Fonts - #55
Replaces the Google Fonts @import in src/styles.css with @fontsource/manrope, imported per weight (400, 500, 600, 700, 800, the same weights as before). The font family is still Manrope, font-display: swap is unchanged, and Vite now bundles the font files into /assets.
Why
When a page loads a font from Google Fonts, the visitor's browser connects to fonts.googleapis.com and fonts.gstatic.com, which sends the visitor's IP address to Google. Serving the font ourselves stops that, so Google no longer has to be listed as a recipient in the privacy policy.
Verification
vp check (format, lint, typecheck) passes
vp test run passes
pnpm run build passes, including the server bundle check
.output contains no fonts.googleapis or fonts.gstatic references, and the Manrope .woff2 files are emitted in .output/public/assets
pnpm install --frozen-lockfile passes. The lockfile diff only adds @fontsource/manrope; the latest dependencies were not re-resolved.
Reviewing files that changed from the base of the PR and between 2a63265 and 654cced.
⛔ Files ignored due to path filters: 1
pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing: 2
package.json
src/styles.css
No actionable comments were generated in the recent review. 🎉
ℹ️ Recent review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Advanced
Run ID: 29fdcb64-e1db-45d3-9a3f-3e13d914e820
📥 Commits
Reviewing files that changed from the base of the PR and between 7ae7521 and 2a63265.
⛔ Files ignored due to path filters: 1
pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing: 2
package.json
src/styles.css
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.
Walkthrough
The project adds @fontsource/manrope and replaces the remote Google Fonts import with package imports for Manrope weights 400, 500, 600, 700, and 800.
package.json adds @fontsource/manrope as a dependency. src/styles.css imports five Manrope weights from the package instead of using a Google Fonts URL.
Priority: ⬇️ Low
Change: Bug fix
Merge Risk:⚪ Minimal · up to 2a632
The change replaces the remote font import with package-based imports; no concrete regression is established, so it appears ready to merge.
Pre-merge checks: 4
Passed checks: 4 passed
Check name
Status
Explanation
Title check
The title clearly and concisely describes the main change: replacing Google Fonts with a self-hosted Manrope font.
Docstring Coverage
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Replaces the Google Fonts
@importinsrc/styles.csswith@fontsource/manrope, imported per weight (400, 500, 600, 700, 800, the same weights as before). The font family is stillManrope,font-display: swapis unchanged, and Vite now bundles the font files into/assets.Why
When a page loads a font from Google Fonts, the visitor's browser connects to
fonts.googleapis.comandfonts.gstatic.com, which sends the visitor's IP address to Google. Serving the font ourselves stops that, so Google no longer has to be listed as a recipient in the privacy policy.Verification
vp check(format, lint, typecheck) passesvp test runpassespnpm run buildpasses, including the server bundle check.outputcontains nofonts.googleapisorfonts.gstaticreferences, and the Manrope.woff2files are emitted in.output/public/assetspnpm install --frozen-lockfilepasses. The lockfile diff only adds@fontsource/manrope; thelatestdependencies were not re-resolved.