Skip to content

fix: remove unused font loading configurations#2455

Merged
jaygiang merged 2 commits intomasterfrom
fix/lcp-font-loading
Feb 19, 2026
Merged

fix: remove unused font loading configurations#2455
jaygiang merged 2 commits intomasterfrom
fix/lcp-font-loading

Conversation

@jaygiang
Copy link
Collaborator

Problem

InterVariable.woff2 is downloaded twice on every page load. The <link rel="preload"> in headTags points to /docs/fonts/Inter/InterVariable.woff2?v=4.0, but Docusaurus/webpack hashes the CSS @font-face URL to /docs/assets/fonts/InterVariable-{hash}.woff2. Since the URLs don't match, the browser treats them as separate resources and downloads both.

Screenshot 2026-02-19 at 10 56 26 AM

Fix

  • Emptied headTags array in docusaurus.config.ts to remove the mismatched font preload
  • Removed unused "Inter var" legacy @font-face declarations from theme.css
  • Removed 20 unused static "Inter" weight @font-face declarations from theme.css
Screenshot 2026-02-19 at 10 52 46 AM

Related Issue or Design Document

Checklist

  • I have read the contributing guidelines and signed the CLA.
  • I have referenced an issue containing the design document if my change introduces a new feature.
  • I have read the security policy.
  • I confirm that this pull request does not address a security vulnerability.
    If this pull request addresses a security vulnerability,
    I confirm that I got approval (please contact security@ory.com) from the maintainers to push the changes.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added the necessary documentation within the code base (if appropriate).

Further comments

References:

@jaygiang jaygiang force-pushed the fix/lcp-font-loading branch from 436d6fd to 30ca2f7 Compare February 19, 2026 20:26
@jaygiang jaygiang merged commit d224f56 into master Feb 19, 2026
10 checks passed
@jaygiang jaygiang deleted the fix/lcp-font-loading branch February 19, 2026 20:48
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