-
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
Noto Serif Balinese: Version 2.006; ttfautohint (v1.8.4.7-5d5b) added #7684
Conversation
Taken from the upstream repo https://github.com/notofonts/balinese at commit notofonts/balinese@e6c2d4c.
FontBakery reportfontbakery version: 0.12.6 Experimental checksThese won't break the CI job for now, but will become effective after some time if nobody raises any concern. [1] NotoSerifBalinese-Regular.ttfAll other checks[14] NotoSerifBalinese-Regular.ttf🔥 FAIL Ensure dotted circle glyph is present and can attach marks.
Use -F or --full-lists to disable shortening of long lists. [code: unattached-dotted-circle-marks]
|
💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
---|---|---|---|---|---|---|---|
0 | 0 | 1 | 8 | 64 | 7 | 171 | 0 |
0% | 0% | 0% | 3% | 25% | 3% | 68% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
|
|
@simoncozens same confirmation about the dotted circle. Other than that, LGTM. |
This is fine again; the Balinese marks attach but Latin ones do not, and the Balinese ones are the important ones here. In the future I'm going to fix ufomerge to merge the anchors on the dotted circle, but for now the font is working as intended. |
Okay. Merging this here now in that case. |
@simoncozens, there seems to be a diacritics clash in Sandbox |
Making a new release now. |
Superseded by #7929 |
Taken from the upstream repo https://github.com/notofonts/balinese at commit notofonts/balinese@e6c2d4c.
PR Checklist: