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

jquery: switch font to self-hosted Cairo #451

Merged
merged 1 commit into from
Jan 4, 2024
Merged

Conversation

timmywil
Copy link
Member

Fixes gh-446

@timmywil timmywil requested a review from Krinkle December 16, 2023 16:55
themes/jquery/css/base.css Outdated Show resolved Hide resolved
themes/jquery/css/base.css Outdated Show resolved Hide resolved
themes/jquery/css/base.css Show resolved Hide resolved
@Krinkle
Copy link
Member

Krinkle commented Dec 18, 2023

@timmywil Can you mention somewhere the origin of the woff file? This make it easier to verify the source of the binary file in the future.

Perhaps also add a license file in the Cairo subdirectory, which would include the copyright attribution statement (assuming the source has one). Afaik the OFL is similar to MIT and CC-BY licenses, in so far that it allows free distribution so long as you attribute the author. Right now, the repo doesn't.

@timmywil
Copy link
Member Author

timmywil commented Dec 20, 2023

File size comparisons

Font ttf woff woff2
Cairo-Bold 93kb 48kb 36kb
Cairo-Regular 93kb 47kb 35kb
FontAwesome 54kb 29kb 24kb

So, a more significant difference from ttf to woff and a relatively minor difference for woff2, but I'll include it now that I have it.

Edit: I went ahead and converted a fontawesome woff2. See above for comparisons.

Copy link
Member

@mgol mgol left a comment

Choose a reason for hiding this comment

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

The PR still adds TTF files. However, this makes sense as we need to allow reproducing how we created the WOFF ones. LGTM!

@Krinkle Krinkle merged commit cdbfde9 into jquery:main Jan 4, 2024
5 checks passed
Krinkle added a commit that referenced this pull request Jan 4, 2024
Set the amount via line-height to match the previous height exactly,
rather than approximating via padding.

Ref #451
Krinkle added a commit that referenced this pull request Jan 4, 2024
Krinkle added a commit that referenced this pull request Jan 4, 2024
Follows-up #451

* URLs were updated to HTTPS at <https://fortawesome.github.com/Font-Awesome>,
  but this has been a 404 Not Found for a while.

  It looks like we currently include version 3.0.2, I've now documented
  this and updated the source URL to https://fontawesome.com/v3/ for
  future reference.

* Move the (unused) font-awesome.css file to the FontAwesome directory,
  and move said directory to lib/ for clarity.

* Fix incorrect license claim from CC BY 3.0 to MIT License for
  Font Awesome CSS declarations included in base.css. As per the
  license header in font-awesome.css, and per <https://fontawesome.com/v3/license/>,
  font is under OFL, and the CSS code under MIT. Only the doc website
  was under CC BY, not the software itself.

* Similarly move Cairo to lib/ and add its entry to NOTICE.txt.

* Preserve a copy of the unmodified normalize.css, along with its
  LICENSE file and the author attribution in the license's copyright
  statement that the MIT license requires to be preserved.
Krinkle added a commit that referenced this pull request Jan 4, 2024
Follows-up #451, to
avoid cached CSS that references font families that the current HTML
no longer loads, and are thus missing.
@timmywil timmywil deleted the cairo branch January 20, 2024 16:32
Krinkle added a commit that referenced this pull request Apr 13, 2024
Krinkle added a commit that referenced this pull request Apr 13, 2024
…choose-gifts, big-message

No search results for these. Mainly motivated by removing old
references to the now-undefined Klavika font.

Ref #451.
Krinkle added a commit that referenced this pull request Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Improve font-loading performance (avoid Typekit JS)
3 participants