Skip to content

host: Remove font links that are also self-hosted#4366

Open
backspace wants to merge 2 commits intomainfrom
self-host-fonts-cs-10706
Open

host: Remove font links that are also self-hosted#4366
backspace wants to merge 2 commits intomainfrom
self-host-fonts-cs-10706

Conversation

@backspace
Copy link
Copy Markdown
Contributor

fonts.css in boxel-ui already includes these.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

Preview deployments

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

Host Test Results

2 185 tests  ±0   2 170 ✅ ±0   2h 19m 17s ⏱️ - 1m 33s
    1 suites ±0      15 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 5a73077. ± Comparison against base commit 861c03b.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

Realm Server Test Results

  1 files  ±0    1 suites  ±0   14m 49s ⏱️ +38s
836 tests ±0  836 ✅ ±0  0 💤 ±0  0 ❌ ±0 
907 runs  ±0  907 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 5a73077. ± Comparison against base commit 861c03b.

♻️ This comment has been updated with latest results.

@backspace backspace marked this pull request as ready for review April 9, 2026 01:02
@backspace backspace requested a review from a team April 9, 2026 01:31
Comment on lines -22 to -23
<link rel="preconnect" href="https://fonts.googleapis.com" />
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin />
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would have guessed these provide performance improvements for loading fonts from Google Fonts. How do you know they are not helpful?

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