-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Linefont: Version 3.002;gftools[0.9.33] added #6799
Conversation
* Linefont Version 3.002;gftools[0.9.33] taken from the upstream repo https://github.com/dy/linefont at commit dy/linefont@347effe.
FontBakery reportfontbakery version: 0.9.2 [1] Family checksℹ INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)
[23] Linefont[wdth,wght].ttf🔥 FAIL: Check OFL body text is correct. (com.google.fonts/check/license/OFL_body_text)
🔥 FAIL: METADATA.pb font.name field contains font name in right format? (com.google.fonts/check/metadata/valid_name_values)
🔥 FAIL: Check a font's STAT table contains compulsory Axis Values. (com.google.fonts/check/STAT)
🔥 FAIL: Validate STAT particle names and values match the fallback names in GFAxisRegistry. (com.google.fonts/check/STAT/gf_axisregistry)
🔥 FAIL: Checking OS/2 usWinAscent & usWinDescent. (com.google.fonts/check/family/win_ascent_and_descent)
🔥 FAIL: Font contains '.notdef' as its first glyph? (com.google.fonts/check/mandatory_glyphs)
⚠ WARN: DESCRIPTION.en_us.html should end in a linebreak. (com.google.fonts/check/description/eof_linebreak)
⚠ WARN: Check for codepoints not covered by METADATA subsets. (com.google.fonts/check/metadata/unreachable_subsetting)
Use -F or --full-lists to disable shortening of long lists. Or you can add the above codepoints to one of the subsets supported by the font: ⚠ WARN: A static fonts directory with at least two fonts must accompany variable fonts (com.google.fonts/check/repo/vf_has_static_fonts)
⚠ WARN: Ensure variable fonts include an avar table. (com.google.fonts/check/mandatory_avar_table)
⚠ WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)
⚠ WARN: Detect any interpolation issues in the font. (com.google.fonts/check/interpolation_issues)
Use -F or --full-lists to disable shortening of long lists. [code: interpolation-issues] ⚠ WARN: Check accent of Lcaron, dcaron, lcaron, tcaron (derived from com.google.fonts/check/alt_caron) (com.google.fonts/check/alt_caron)
⚠ WARN: Checking correctness of monospaced metadata. (com.google.fonts/check/monospace)
⚠ WARN: Are there any misaligned on-curve points? (com.google.fonts/check/outline_alignment_miss)
Use -F or --full-lists to disable shortening of long lists. [code: found-misalignments] ℹ INFO: Does DESCRIPTION file contain a upstream Git repo URL? (com.google.fonts/check/description/git_url)
ℹ INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)
ℹ INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)
ℹ INFO: EPAR table present in font? (com.google.fonts/check/epar)
ℹ INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)
PPM <= 65535: ℹ INFO: Check for font-v versioning. (com.google.fonts/check/fontv)
ℹ INFO: Font contains all required tables? (com.google.fonts/check/required_tables)
ℹ INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)
Summary
Note: The following loglevels were omitted in this report:
|
🔥 FAIL: Check OFL body text is correct. (com.google.fonts/check/license/OFL_body_text) The difference is an empty line at the end of the doc and a space at the end of a line. So can be ignored, reported here: fonttools/fontbakery#4289 🔥 FAIL: METADATA.pb font.name field contains font name in right format? (com.google.fonts/check/metadata/valid_name_values) Bug. reported here: fonttools/fontbakery#4262 🔥 FAIL: Check a font's STAT table contains compulsory Axis Values. (com.google.fonts/check/STAT) Yelling because of ExtraBlack in the STAT table, which accept so far as long as not existing as an instance in the FVAR table. But we also found another issue with @vv-monsalve, that SuperCondensed maybe shouldn't be in the Axis Registry and so also not in the STAT of Line fonts. So we are waiting for this to get resolved: googlefonts/googlefonts.github.io#113 🔥 FAIL: Checking OS/2 usWinAscent & usWinDescent. (com.google.fonts/check/family/win_ascent_and_descent) These are already explain in Wavefont; special font, we made an exception for both Families that work together. Cf #6705 (comment) We may also need to merge to see what it looks like in dev-sandbox, and amend the PR with a custom sample text like for Wavefont. |
The discussion around axis registry consistency is still ongoing, and I think could take a little while. So, if the |
@RosaWagner The zip file indeed includes the |
dfef2e4: [gftools-packager] Linefont: Version 3.002;gftools[0.9.33] added
a67abd1: Linefont: updated metadata.pb and description
Fixes #4630