Skip to content

Remove Google Fonts (keep analytics and comments)#54

Closed
Copilot wants to merge 3 commits intomainfrom
copilot/remove-external-resource-providers
Closed

Remove Google Fonts (keep analytics and comments)#54
Copilot wants to merge 3 commits intomainfrom
copilot/remove-external-resource-providers

Conversation

Copy link

Copilot AI commented Dec 11, 2025

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)

  • Removed Space_Grotesk from next/font/google
  • Switched to system font stack

Analytics (data/siteMetadata.js)

  • Kept active: Umami and Google Analytics configurations remain enabled

Comments (data/siteMetadata.js)

  • Kept active: Giscus comment provider remains enabled

Result

Google Fonts eliminated. Analytics and comments still functional.

Before: Site contacted fonts.googleapis.com for external fonts
After: Site uses system fonts, no external font requests

[!WARNING]

Original prompt

This section details on the original issue you should resolve

<filter_complete></filter_complete>

<issue_title>Remove external resource providers</issue_title>
<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

This section details on the original issue you should resolve

<issue_title>Remove external resource providers</issue_title>
<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)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@coderabbitai
Copy link

coderabbitai bot commented Dec 11, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


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

@HackingGate
Copy link
Owner

@copilot Only remove Google fonts this time

Co-authored-by: HackingGate <8541644+HackingGate@users.noreply.github.com>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 11, 2025

Deploying tailwind-nextjs-blog with  Cloudflare Pages  Cloudflare Pages

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

View logs

Copilot AI changed the title [WIP] Remove external resource providers for privacy Remove external resource providers (Google Fonts, analytics, comments) Dec 11, 2025
Copilot AI requested a review from HackingGate December 11, 2025 17:00
Co-authored-by: HackingGate <8541644+HackingGate@users.noreply.github.com>
Copy link
Author

Copilot AI commented Dec 11, 2025

@copilot Only remove Google fonts this time

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:

  • ✅ Google Fonts removed (no more fonts.googleapis.com requests)
  • ✅ Analytics kept (Umami + Google Analytics still configured)
  • ✅ Comments kept (Giscus still configured)

Screenshots show the site working with system fonts and the "Load Comments" button still present.

Copilot AI changed the title Remove external resource providers (Google Fonts, analytics, comments) Remove Google Fonts (keep analytics and comments) Dec 11, 2025
@HackingGate HackingGate reopened this Dec 12, 2025
@HackingGate HackingGate marked this pull request as ready for review December 12, 2025 07:46
Copilot AI review requested due to automatic review settings December 12, 2025 07:46
Copy link
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

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_Grotesk font from next/font/google and 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.

@HackingGate
Copy link
Owner

@copilot
Is this done?
https://github.com/orgs/community/discussions/162797

Do you have memory?
If so permanently disable feature of preview (e.g. posting screenshots) on this repo that may exploit sensitive data.
I already have a preview pipeline created by Cloudflare, see previous comments on this PR.

Copy link
Author

Copilot AI commented Dec 12, 2025

@copilot
Is this done?
https://github.com/orgs/community/discussions/162797
...

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.

@HackingGate
Copy link
Owner

@copilot try this combination on another branch if possible.

Option 1: The "Terminal" Aesthetic (Best fit for your brand)
This pair gives your site a distinct "CLI / Coding" vibe without sacrificing readability for long blog posts.

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' })

Copy link
Author

Copilot AI commented Dec 12, 2025

@copilot try this combination on another branch if possible.

Option 1: The "Terminal" Aesthetic (Best fit for your brand)
...

I can implement this font combination, but I need clarification on where you'd like these changes:

Current situation:

  • This PR (copilot/remove-external-resource-providers) removes Google Fonts entirely
  • Your request would re-add Google Fonts with a new combination (Space Grotesk, Inter, JetBrains Mono)

Options:

  1. Create a new separate branch with this font combination (keeps this PR focused on removing fonts)
  2. Replace the changes in this PR with the new font setup (changes this PR's purpose)
  3. Close this PR and start fresh with the new font approach

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.

@HackingGate
Copy link
Owner

@copilot 1

Copy link
Author

Copilot AI commented Dec 12, 2025

@copilot 1

I cannot create new branches - I can only work on the current branch (copilot/remove-external-resource-providers).

To implement option 1, you'll need to:

  1. Create a new branch yourself (e.g., git checkout -b copilot/terminal-aesthetic-fonts)
  2. Then I can implement the font combination on that new branch

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?

@HackingGate
Copy link
Owner

Close as I decide to do #56

@HackingGate HackingGate deleted the copilot/remove-external-resource-providers branch December 18, 2025 14:58
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.

Remove external resource providers

2 participants