Skip to content
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

Trim fonts flavors to TTF/OTF only #714

Closed
wants to merge 3 commits into from

Conversation

xavierba
Copy link
Contributor

Nowadays, TTF and OTF font formats are supported in all browsers:
https://caniuse.com/#search=ttf

Subsequently, there's no need to clutter the distribution with font flavors that won't be used anyway.
Removing all but the TTF/OTF flavors for Font-Awesome and Foundation-Icons saves 350+KB.

Please note this change is unlikely to break any browser, as the third font shipped with Sympa, Raleway, is only distributed in the OTF flavor (and was only shipped in TTF flavor before #191).

The third commit is a fix to correct Raleway font format in default css. This supposedly minor bug, as it went unnoticed for more than a year, was introduced in #191.

@ldidry
Copy link
Contributor

ldidry commented Jul 31, 2019

Please, do not trim .woff and .woff2 formats since they are lighter than ttf/otf (in fact, they are compressed ttf/otf). Those formats are the future, not the past like eot (that, as for me, you can remove).

Woff is well supported by browsers: https://caniuse.com/#search=woff
Woff2 is supported by modern browsers: https://caniuse.com/#search=woff2

@ikedas ikedas added the packaging On source distribution and derived packages label Aug 9, 2019
@ikedas
Copy link
Member

ikedas commented Aug 14, 2019

At present we will partially adopt this PR to keep woff/woff2 formats. Close.

@ikedas ikedas closed this Aug 14, 2019
@ikedas ikedas added this to the 6.2.46 milestone Aug 14, 2019
ikedas added a commit that referenced this pull request Aug 14, 2019
@xavierba
Copy link
Contributor Author

xavierba commented Aug 14, 2019

@ikedas , sorry for not updating the PR myself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packaging On source distribution and derived packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants