Skip to content

Drop unused font-awesome fonts & keep only woff2 (we are in 2025 ffs) #556

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Drop unused font-awesome fonts & keep only woff2 (we are in 2025 ffs) #556

wants to merge 3 commits into from

Conversation

phndiaye
Copy link
Member

What does this PR do?

This PR removes two things:

  • unused FontAwesome fonts like Sharp, Thin, or Light that we do not use. If needed they will be added.
  • Remove eot|ttf|woff fonts as woff2 have been supported on baseline web for quite some time now (https://caniuse.com/woff2). Also, those are quite heavy and are only needed when targeting really old browsers.

This allows us the drop the dist/ size quite considerably.

What are the observable changes?

Stage dist/ Size Delta vs Previous Delta vs Initial
Initial (all fonts) 19.0 MB – –
Mid (unused fonts dropped) 9.8 MB −9.2 MB (−48%) −9.2 MB (−48%)
Current (woff2 only) 6.9 MB −2.9 MB (−30%) −12.1 MB (−64%)

Good PR checklist

  • Title makes sense
  • Is against the correct branch
  • Only addresses one issue
  • Properly assigned
  • Added/updated tests
  • Added/updated documentation
  • Migrated touched components to Glimmer Components
  • Properly labeled

@phndiaye phndiaye self-assigned this Aug 20, 2025
@phndiaye phndiaye force-pushed the chore/cleanup-bootstrap-js-import branch from 9309997 to 717f4bc Compare September 12, 2025 15:50
Base automatically changed from chore/cleanup-bootstrap-js-import to master October 1, 2025 09:45
@phndiaye phndiaye force-pushed the chore/drop-fontawesome-weight branch from 9b39607 to 2504237 Compare October 16, 2025 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Miexil Miexil Miexil approved these changes

@OwenCoogan OwenCoogan OwenCoogan approved these changes

@JulienVannier66 JulienVannier66 JulienVannier66 approved these changes

@olxmpe olxmpe olxmpe approved these changes

@aprentout aprentout aprentout approved these changes

Assignees

@phndiaye phndiaye

Labels

reviewed/ready-to-merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@phndiaye @Miexil @OwenCoogan @JulienVannier66 @olxmpe @aprentout @upfluence-bot