-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Playwrite VN: Version 1.001 added #7639
Conversation
Taken from the upstream repo https://github.com/TypeTogether/Playwrite at commit TypeTogether/Playwrite@cf0e1a0.
Taken from the upstream repo https://github.com/TypeTogether/Playwrite at commit TypeTogether/Playwrite@cf0e1a0.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
FontBakery reportfontbakery version: 0.12.5 Experimental checksThese won't break the CI job for now, but will become effective after some time if nobody raises any concern. [1] PlaywriteVN[wght].ttf
|
ofl/playwritevn/PlaywriteVN[wght].ttf | |
---|---|
Dehinted Size | 326.6kb |
Hinted Size | 326.6kb |
Increase | 24 bytes |
Change | 0.0 % |
ℹ️ INFO Ensure fonts have ScriptLangTags declared on the 'meta' table.
-
ℹ️ INFO
Latn,af,ca,cs,cy,da,en,es,et,eu,fr,ga,hi,hr,hu,id,is,it,nl,no,pl,pt,ro,sk,sl,sv,tr,vi,wa,yo,zu
[code: dlng-tag] -
ℹ️ INFO
Latn,af,ca,cs,cy,da,en,es,et,eu,fr,ga,hi,hr,hu,id,is,it,nl,no,pl,pt,ro,sk,sl,sv,tr,vi,wa,yo,zu
[code: slng-tag]
[1] Family checks
Summary
💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
---|---|---|---|---|---|---|---|
0 | 0 | 1 | 7 | 52 | 9 | 181 | 0 |
0% | 0% | 0% | 3% | 21% | 4% | 72% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
This font, as in many other cases within the Playwrite project, has unusual ascenders and descenders that, combined with the Vietnamese support, will also require unusual Vertical Metrics. |
@emmamarichal I need to check how the images are working on the Article, so I'll merge this one now to inspect it in dev. |
Superseded by #7668. I'm removing this one from the Traffic board now. |
Taken from the upstream repo https://github.com/TypeTogether/Playwrite at commit TypeTogether/Playwrite@cf0e1a0.
PR Checklist: