Remove Google Fonts (keep analytics and comments)#54
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
|
@copilot Only remove Google fonts this time |
Co-authored-by: HackingGate <8541644+HackingGate@users.noreply.github.com>
Deploying tailwind-nextjs-blog with
|
| Latest commit: |
9b05e10
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://b9228a2a.tailwind-nextjs-blog.pages.dev |
| Branch Preview URL: | https://copilot-remove-external-reso.tailwind-nextjs-blog.pages.dev |
Co-authored-by: HackingGate <8541644+HackingGate@users.noreply.github.com>
Done! Reverted the analytics and comments changes in commit 9b05e10. Now only Google Fonts have been removed - the site uses system fonts instead. What changed:
Screenshots show the site working with system fonts and the "Load Comments" button still present. |
There was a problem hiding this comment.
Pull request overview
This PR successfully removes Google Fonts from the site to eliminate external font loading from fonts.googleapis.com, switching to a system font stack while preserving analytics and comment functionality.
Key Changes:
- Removed
Space_Groteskfont fromnext/font/googleand switched to system fonts - Maintained active Umami Analytics, Google Analytics, and Giscus comments configurations
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
app/layout.tsx |
Removed Space_Grotesk font import, configuration, and className variable reference |
css/tailwind.css |
Updated --font-sans variable to use only system fonts without Google Fonts fallback |
The implementation is clean and complete. All references to the Space_Grotesk font have been successfully removed, and the font stack now relies entirely on system fonts (ui-sans-serif, system-ui, etc.). The changes accomplish the stated goal of eliminating external resource loading for fonts while keeping analytics and comment systems functional. No issues were identified with the implementation.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot Do you have memory? |
Yes, the Google Fonts removal is complete. The PR successfully removes external font loading while keeping analytics and comments active. Regarding memory and screenshot previews: I don't have persistent memory across sessions - each conversation starts fresh. I cannot permanently disable features for a repository. Screenshot generation during my work is part of the standard workflow to show UI changes, but I don't control this feature's configuration. For deployment previews, you mentioned having a Cloudflare pipeline which you control directly. If you want to disable automatic GitHub deployment previews or Actions that might expose data, you'd need to configure that in your repository settings or GitHub Actions workflows. |
|
@copilot try this combination on another branch if possible. Option 1: The "Terminal" Aesthetic (Best fit for your brand) Headings: Space Grotesk (A quirky, geometric sans that feels "techy") Body: Inter (Standard, highly legible) or Space Mono (If you want to go full hacker, though it can be harder to read in long paragraphs) Code Blocks: JetBrains Mono (The gold standard for code) import { Space_Grotesk, Inter, JetBrains_Mono } from 'next/font/google'
const spaceGrotesk = Space_Grotesk({ subsets: ['latin'], variable: '--font-heading' })
const inter = Inter({ subsets: ['latin'], variable: '--font-body' })
const jetbrains = JetBrains_Mono({ subsets: ['latin'], variable: '--font-code' }) |
I can implement this font combination, but I need clarification on where you'd like these changes: Current situation:
Options:
Which approach would you prefer? I recommend option 1 (new branch) to keep this PR's purpose clear, but I'll need you to specify the branch name you'd like me to use. |
|
@copilot 1 |
I cannot create new branches - I can only work on the current branch ( To implement option 1, you'll need to:
Alternative: I can provide you with the exact code changes you need to make, and you can apply them on a new branch yourself. Would either of these approaches work for you? |
|
Close as I decide to do #56 |
Removes Google Fonts to eliminate external font loading from
fonts.googleapis.com, while keeping analytics and comment system configurations active.Changes
Font Loading (
app/layout.tsx,css/tailwind.css)Space_Groteskfromnext/font/googleAnalytics (
data/siteMetadata.js)Comments (
data/siteMetadata.js)Result
Google Fonts eliminated. Analytics and comments still functional.
Before: Site contacted
fonts.googleapis.comfor external fontsAfter: Site uses system fonts, no external font requests
Original prompt
<issue_description>Check all external resources loaded when opening the site that might exploit privacy.</issue_description>
Comments on the Issue (you are @copilot in this section)
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.