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

WOFF font support #7449

Open
wants to merge 18 commits into
base: dev-2.0
Choose a base branch
from
Open

WOFF font support #7449

wants to merge 18 commits into from

Conversation

dhowe
Copy link
Contributor

@dhowe dhowe commented Dec 27, 2024

  • Add WOFF support via upgraded Typr.js
  • Fixes null argument bug in fontWidth/textWidth
  • Fixes direct-font-set triggering on non-string fonts

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like the test for this still exists, were these meant to be deleted? If not, we can restore by running this then committing the results:

git checkout dev-2.0 -- "test/unit/visual/screenshots/Typography/textWeight/can control variable fonts from files"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

tests still failing after I restored those files, but now with a pixel mismatch -- I regenerated the screenshots but still needs a visual check

Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like the font has changed now, so I think pixel mismatch failures were actually correct in this instance, and something about font loading has changed:
image

package-lock.json Outdated Show resolved Hide resolved
@dhowe dhowe changed the title Woff support WOFF font support Dec 28, 2024
@davepagurek
Copy link
Contributor

@limzykenneth looks like we just merged some FES changes into dev-2.0, I think they might be causing some test failures
image

@limzykenneth
Copy link
Member

Yeah still fixing it, the test for FES may need to be rewritten a bit and some outdated documentation around parameters may also need to be updated. I'll push fixes some time next week.

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.

3 participants