-
Notifications
You must be signed in to change notification settings - Fork 5
Replace Google Fonts CDN with self-hosted Inter font files #47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: garland3 <[email protected]>
There was a problem hiding this 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 pull request replaces the external Google Fonts import with self-hosted Inter font files to improve performance, privacy, and reliability by eliminating external dependencies.
Key changes:
- Removed Google Fonts CDN dependency for Inter font family
- Added self-hosted Inter font files in woff2 format (Regular, Medium, SemiBold, Bold)
- Updated CSS to reference local font files using @font-face declarations
Reviewed Changes
Copilot reviewed 1 out of 5 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| frontend/src/index.css | Replaced Google Fonts import with @font-face declarations for self-hosted fonts |
| frontend/public/fonts/Inter-SemiBold.woff2 | Added SemiBold weight variant of Inter font (binary file) |
…ile processing - Remove large block of commented-out system status and MCP health endpoints from admin_routes.py - Disable file processing in websocket_update_callback for canvas_files and files_update types in main.py, replacing with pass statements This commit streamlines the codebase by eliminating unused commented code and temporarily disabling non-essential file update handling in websocket callbacks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 5 out of 10 changed files in this pull request and generated 2 comments.
backend/main.py
Outdated
| pass | ||
| # files = (message.get("data") or {}).get("files") or [] |
Copilot
AI
Nov 3, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Empty pass statements with commented code create ambiguity. Either remove the commented code entirely or document why it's being preserved for future use with a TODO/NOTE comment explaining the intent.
backend/main.py
Outdated
| pass | ||
| # files = (message.get("data") or {}).get("files") or [] |
Copilot
AI
Nov 3, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent indentation on the commented line. The comment starts at column 0 but should align with the surrounding code block indentation for readability.
…g commented logging - Consolidated handling for 'canvas_files' and 'files_update' types - Removed disabled logging code to reduce noise and improve readability - Preserves functionality while cleaning up the websocket callback logic
The frontend was loading Inter font from Google Fonts CDN, violating CSP
style-src 'self'and breaking CSS rendering on admin pages.Changes
frontend/src/index.cssfrontend/public/fonts/@font-facedeclarations referencing local filesBefore
After
Screenshot
Fonts now load from same origin, eliminating CSP violations and restoring proper text rendering.
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
astral.shcurl -LsSf REDACTED(dns block)If you need me to access, download, or install something from one of these locations, you can either:
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.