-
Notifications
You must be signed in to change notification settings - Fork 2.7k
XL Lubrifont TC: 2.100 added #9291
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
Conversation
Taken from the upstream repo https://github.com/NightFurySL2001/WD-XL-font at commit https://github.com/NightFurySL2001/WD-XL-font/tree/75c4a3bb2fffefa0f0aae470c2c0a73d6f79d0b9
FontBakery reportfontbakery version: 0.13.2 Check results[33] WD-XLLubrifontTC-Regular.ttf🔥 FAIL Ensure files are not too large. file_size
🔥 FAIL Check font family directory name. googlefonts/metadata/family_directory_name
🔥 FAIL METADATA.pb font.full_name and font.post_script_name fields have equivalent values ? googlefonts/metadata/match_fullname_postscript
🔥 FAIL METADATA.pb font.name and font.full_name fields match the values declared on the name table? googlefonts/metadata/nameid/family_and_full_names
🔥 FAIL METADATA.pb font.post_script_name field contains font name in right format? googlefonts/metadata/valid_post_script_name_values
🔥 FAIL Check family name for GF Guide compliance. googlefonts/family_name_compliance
🔥 FAIL Check copyright namerecords match license file. googlefonts/name/license
🔥 FAIL Check font follows the Google Fonts CJK vertical metric schema googlefonts/cjk_vertical_metrics
|
ofl/wdxllubrifonttc/WD-XLLubrifontTC-Regular.ttf | |
---|---|
Dehinted Size | 9.9Mb |
Hinted Size | 9.9Mb |
Increase | 24 bytes |
Change | 0.0 % |
ℹ️ INFO Font contains all required tables? required_tables
- ℹ️ INFO
This font contains the following optional tables:
- loca
- prep
- BASE
- GPOS
- GSUB
- gasp
- vhea
- vmtx
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field. googlefonts/metadata/minisite_url
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? googlefonts/gasp
- ℹ️ INFO
These are the ppm ranges declared on the gasp table:
PPM <= 65535: flag = 0x0F - Use grid-fitting - Use grayscale rendering - Use gridfitting with ClearType symmetric smoothing - Use smoothing along multiple axes with ClearType®
[code: ranges]ℹ️ INFO Font has old ttfautohint applied? googlefonts/old_ttfautohint
- ℹ️ INFO
Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 2.001;hotconv 1.1.1;makeotfexe 2.6.0']
[code: version-not-detected]
[2] Family checks
🔥 FAIL Directory name in GFonts repo structure must match NameID 1 of the regular. googlefonts/repo/dirname_matches_nameid_1
- 🔥 FAIL
Family name on the name table ('WD-XL 滑油字 TC') does not match directory name in the repo structure ('wdxllubrifonttc'). Expected 'wdxl滑油字tc'.
[code: mismatch]
ℹ️ INFO Check axis ordering on the STAT table. googlefonts/STAT/axis_order
- ℹ️ INFO
All of the fonts lack a STAT table.
[code: summary]
Summary
💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
---|---|---|---|---|---|---|---|
0 | 0 | 9 | 19 | 74 | 7 | 127 | 0 |
0% | 0% | 4% | 8% | 31% | 3% | 54% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
FontBakery reportfontbakery version: 0.13.2 Check results[32] WD-XLLubrifontTC-Regular.ttf🔥 FAIL Ensure files are not too large. file_size
🔥 FAIL METADATA.pb font.full_name and font.post_script_name fields have equivalent values ? googlefonts/metadata/match_fullname_postscript
🔥 FAIL METADATA.pb font.name and font.full_name fields match the values declared on the name table? googlefonts/metadata/nameid/family_and_full_names
🔥 FAIL METADATA.pb font.post_script_name field contains font name in right format? googlefonts/metadata/valid_post_script_name_values
🔥 FAIL Check family name for GF Guide compliance. googlefonts/family_name_compliance
🔥 FAIL Check copyright namerecords match license file. googlefonts/name/license
🔥 FAIL Check font follows the Google Fonts CJK vertical metric schema googlefonts/cjk_vertical_metrics
|
ofl/xllubrifonttc/WD-XLLubrifontTC-Regular.ttf | |
---|---|
Dehinted Size | 9.9Mb |
Hinted Size | 9.9Mb |
Increase | 24 bytes |
Change | 0.0 % |
ℹ️ INFO Font contains all required tables? required_tables
- ℹ️ INFO
This font contains the following optional tables:
- loca
- prep
- BASE
- GPOS
- GSUB
- gasp
- vhea
- vmtx
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field. googlefonts/metadata/minisite_url
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? googlefonts/gasp
- ℹ️ INFO
These are the ppm ranges declared on the gasp table:
PPM <= 65535: flag = 0x0F - Use grid-fitting - Use grayscale rendering - Use gridfitting with ClearType symmetric smoothing - Use smoothing along multiple axes with ClearType®
[code: ranges]ℹ️ INFO Font has old ttfautohint applied? googlefonts/old_ttfautohint
- ℹ️ INFO
Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 2.001;hotconv 1.1.1;makeotfexe 2.6.0']
[code: version-not-detected]
[2] Family checks
🔥 FAIL Directory name in GFonts repo structure must match NameID 1 of the regular. googlefonts/repo/dirname_matches_nameid_1
- 🔥 FAIL
Family name on the name table ('WD-XL 滑油字 TC') does not match directory name in the repo structure ('xllubrifonttc'). Expected 'wdxl滑油字tc'.
[code: mismatch]
ℹ️ INFO Check axis ordering on the STAT table. googlefonts/STAT/axis_order
- ℹ️ INFO
All of the fonts lack a STAT table.
[code: summary]
Summary
💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
---|---|---|---|---|---|---|---|
0 | 0 | 8 | 19 | 74 | 7 | 128 | 0 |
0% | 0% | 3% | 8% | 31% | 3% | 54% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
Fixing license string
FontBakery reportfontbakery version: 0.13.2 Check results[32] WD-XLLubrifontTC-Regular.ttf🔥 FAIL Ensure files are not too large. file_size
🔥 FAIL METADATA.pb font.full_name and font.post_script_name fields have equivalent values ? googlefonts/metadata/match_fullname_postscript
🔥 FAIL METADATA.pb font.name and font.full_name fields match the values declared on the name table? googlefonts/metadata/nameid/family_and_full_names
🔥 FAIL METADATA.pb font.post_script_name field contains font name in right format? googlefonts/metadata/valid_post_script_name_values
🔥 FAIL Check family name for GF Guide compliance. googlefonts/family_name_compliance
🔥 FAIL Check copyright namerecords match license file. googlefonts/name/license
🔥 FAIL Check font follows the Google Fonts CJK vertical metric schema googlefonts/cjk_vertical_metrics
|
ofl/xllubrifonttc/WD-XLLubrifontTC-Regular.ttf | |
---|---|
Dehinted Size | 9.9Mb |
Hinted Size | 9.9Mb |
Increase | 24 bytes |
Change | 0.0 % |
ℹ️ INFO Font contains all required tables? required_tables
- ℹ️ INFO
This font contains the following optional tables:
- loca
- prep
- BASE
- GPOS
- GSUB
- gasp
- vhea
- vmtx
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field. googlefonts/metadata/minisite_url
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? googlefonts/gasp
- ℹ️ INFO
These are the ppm ranges declared on the gasp table:
PPM <= 65535: flag = 0x0F - Use grid-fitting - Use grayscale rendering - Use gridfitting with ClearType symmetric smoothing - Use smoothing along multiple axes with ClearType®
[code: ranges]ℹ️ INFO Font has old ttfautohint applied? googlefonts/old_ttfautohint
- ℹ️ INFO
Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 2.001;hotconv 1.1.1;makeotfexe 2.6.0']
[code: version-not-detected]
[2] Family checks
🔥 FAIL Directory name in GFonts repo structure must match NameID 1 of the regular. googlefonts/repo/dirname_matches_nameid_1
- 🔥 FAIL
Family name on the name table ('WD-XL 滑油字 TC') does not match directory name in the repo structure ('xllubrifonttc'). Expected 'wdxl滑油字tc'.
[code: mismatch]
ℹ️ INFO Check axis ordering on the STAT table. googlefonts/STAT/axis_order
- ℹ️ INFO
All of the fonts lack a STAT table.
[code: summary]
Summary
💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
---|---|---|---|---|---|---|---|
0 | 0 | 8 | 19 | 73 | 7 | 129 | 0 |
0% | 0% | 3% | 8% | 31% | 3% | 55% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
I wonder how many things did the dash broke to had to drop "WD-" here 🤔 |
@NightFurySL2001 Oh, I just handwrote that. Don't recall why I didn't put the "WD-" in there. It is still in the font. |
I see. Metadata.pb also had the name dropped too so might need a fix. |
Done |
FontBakery reportfontbakery version: 0.13.2 Check results[33] WD-XLLubrifontTC-Regular.ttf🔥 FAIL Ensure files are not too large. file_size
🔥 FAIL Check font family directory name. googlefonts/metadata/family_directory_name
🔥 FAIL METADATA.pb font.full_name and font.post_script_name fields have equivalent values ? googlefonts/metadata/match_fullname_postscript
🔥 FAIL METADATA.pb font.name and font.full_name fields match the values declared on the name table? googlefonts/metadata/nameid/family_and_full_names
🔥 FAIL METADATA.pb font.post_script_name field contains font name in right format? googlefonts/metadata/valid_post_script_name_values
🔥 FAIL Check family name for GF Guide compliance. googlefonts/family_name_compliance
🔥 FAIL Check copyright namerecords match license file. googlefonts/name/license
🔥 FAIL Check font follows the Google Fonts CJK vertical metric schema googlefonts/cjk_vertical_metrics
|
ofl/xllubrifonttc/WD-XLLubrifontTC-Regular.ttf | |
---|---|
Dehinted Size | 9.9Mb |
Hinted Size | 9.9Mb |
Increase | 24 bytes |
Change | 0.0 % |
ℹ️ INFO Font contains all required tables? required_tables
- ℹ️ INFO
This font contains the following optional tables:
- loca
- prep
- BASE
- GPOS
- GSUB
- gasp
- vhea
- vmtx
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field. googlefonts/metadata/minisite_url
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? googlefonts/gasp
- ℹ️ INFO
These are the ppm ranges declared on the gasp table:
PPM <= 65535: flag = 0x0F - Use grid-fitting - Use grayscale rendering - Use gridfitting with ClearType symmetric smoothing - Use smoothing along multiple axes with ClearType®
[code: ranges]ℹ️ INFO Font has old ttfautohint applied? googlefonts/old_ttfautohint
- ℹ️ INFO
Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 2.001;hotconv 1.1.1;makeotfexe 2.6.0']
[code: version-not-detected]
[2] Family checks
🔥 FAIL Directory name in GFonts repo structure must match NameID 1 of the regular. googlefonts/repo/dirname_matches_nameid_1
- 🔥 FAIL
Family name on the name table ('WD-XL 滑油字 TC') does not match directory name in the repo structure ('xllubrifonttc'). Expected 'wdxl滑油字tc'.
[code: mismatch]
ℹ️ INFO Check axis ordering on the STAT table. googlefonts/STAT/axis_order
- ℹ️ INFO
All of the fonts lack a STAT table.
[code: summary]
Summary
💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
---|---|---|---|---|---|---|---|
0 | 0 | 9 | 19 | 73 | 7 | 128 | 0 |
0% | 0% | 4% | 8% | 31% | 3% | 54% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
FontBakery reportfontbakery version: 0.13.2 Check results[33] WD-XLLubrifontTC-Regular.ttf🔥 FAIL Ensure files are not too large. file_size
🔥 FAIL Check font family directory name. googlefonts/metadata/family_directory_name
🔥 FAIL METADATA.pb font.full_name and font.post_script_name fields have equivalent values ? googlefonts/metadata/match_fullname_postscript
🔥 FAIL METADATA.pb font.name and font.full_name fields match the values declared on the name table? googlefonts/metadata/nameid/family_and_full_names
🔥 FAIL METADATA.pb font.post_script_name field contains font name in right format? googlefonts/metadata/valid_post_script_name_values
🔥 FAIL Check family name for GF Guide compliance. googlefonts/family_name_compliance
🔥 FAIL Check copyright namerecords match license file. googlefonts/name/license
🔥 FAIL Check font follows the Google Fonts CJK vertical metric schema googlefonts/cjk_vertical_metrics
|
ofl/wdxllubrifonttc/WD-XLLubrifontTC-Regular.ttf | |
---|---|
Dehinted Size | 9.9Mb |
Hinted Size | 9.9Mb |
Increase | 24 bytes |
Change | 0.0 % |
ℹ️ INFO Font contains all required tables? required_tables
- ℹ️ INFO
This font contains the following optional tables:
- loca
- prep
- BASE
- GPOS
- GSUB
- gasp
- vhea
- vmtx
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field. googlefonts/metadata/minisite_url
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? googlefonts/gasp
- ℹ️ INFO
These are the ppm ranges declared on the gasp table:
PPM <= 65535: flag = 0x0F - Use grid-fitting - Use grayscale rendering - Use gridfitting with ClearType symmetric smoothing - Use smoothing along multiple axes with ClearType®
[code: ranges]ℹ️ INFO Font has old ttfautohint applied? googlefonts/old_ttfautohint
- ℹ️ INFO
Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 2.001;hotconv 1.1.1;makeotfexe 2.6.0']
[code: version-not-detected]
[2] Family checks
🔥 FAIL Directory name in GFonts repo structure must match NameID 1 of the regular. googlefonts/repo/dirname_matches_nameid_1
- 🔥 FAIL
Family name on the name table ('WD-XL 滑油字 TC') does not match directory name in the repo structure ('wdxllubrifonttc'). Expected 'wdxl滑油字tc'.
[code: mismatch]
ℹ️ INFO Check axis ordering on the STAT table. googlefonts/STAT/axis_order
- ℹ️ INFO
All of the fonts lack a STAT table.
[code: summary]
Summary
💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
---|---|---|---|---|---|---|---|
0 | 0 | 9 | 19 | 73 | 7 | 128 | 0 |
0% | 0% | 4% | 8% | 31% | 3% | 54% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
@NightFurySL2001 So.... it turns out the hyphen does in fact break everything. Sigh. |
I think without space is better, WDXL. (This assumption might need to be check in detail in the future though) |
Taken from the upstream repo https://github.com/NightFurySL2001/WD-XL-font at commit https://github.com/NightFurySL2001/WD-XL-font/tree/75c4a3bb2fffefa0f0aae470c2c0a73d6f79d0b9
Font is 10.3MB, single weight.