search

google / fonts

Font files available from Google Fonts, and a public issue tracker for all things Google Fonts
https://fonts.google.com
17.83k stars 2.6k forks source link

Laura Garcia Mut designer info #7850

Closed yanone closed 3 weeks ago

yanone commented 3 weeks ago
github-actions[bot] commented 3 weeks ago

FontBakery report

fontbakery version: 0.12.7

Experimental checks

These won't break the CI job for now, but will become effective after some time if nobody raises any concern.

[2] Fustat[wght].ttf
⚠️ WARN Validate size, and resolution of article images, and ensure article page has minimum length and includes visual assets.
* ⚠️ **WARN**

Family metadata at ofl/fustat does not have an article.

[code: lacks-article]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field.
* ℹ️ **INFO**

Please consider adding a family.minisite_url entry.

[code: lacks-minisite-url]

All other checks

[16] Fustat[wght].ttf
⚠️ WARN Check font contains no unreachable glyphs
* ⚠️ **WARN**

The following glyphs could not be reached by codepoint or substitution rules:

- doubleverticalbarbelowar

- uni0307.i

- uni0308.narrow
[code: unreachable-glyphs]
⚠️ WARN Glyph names are all valid?
* ⚠️ **WARN**

The following glyph names may be too long for some legacy systems which may expect a maximum 31-characters length limit: twodotshorizontalabove_tahabovear, twodotshorizontalbelow_tahabovear and twodotshorizontalcenter_tahabovear

[code: legacy-long-names]
⚠️ WARN Are there any misaligned on-curve points?
* ⚠️ **WARN**

The following glyphs have on-curve points which have potentially incorrect y coordinates:

* g (U+0067): X=404.5,Y=-2.0 (should be at baseline 0?)

* g (U+0067): X=419.5,Y=501.5 (should be at x-height 500?)

* gbreve (U+011F): X=404.5,Y=-2.0 (should be at baseline 0?)

* uni0123 (U+0123): X=404.5,Y=-2.0 (should be at baseline 0?)

* gdotaccent (U+0121): X=404.5,Y=-2.0 (should be at baseline 0?)

* uni0627.fina: X=70.0,Y=1.0 (should be at baseline 0?)

* uni0627.short.fina: X=70.0,Y=1.0 (should be at baseline 0?)

* uni08C0 (U+08C0): X=382.0,Y=701.0 (should be at cap-height 700?)

* uni08C0 (U+08C0): X=428.0,Y=701.0 (should be at cap-height 700?)

* uni08C0 (U+08C0): X=489.0,Y=701.0 (should be at cap-height 700?)

* 61 more.

Use -F or --full-lists to disable shortening of long lists.

[code: found-misalignments]
⚠️ WARN METADATA.pb: Designers are listed correctly on the Google Fonts catalog?
* ⚠️ **WARN**

It seems that Laura Garcia Mut is still not listed on the designers catalog. Please submit a photo and a link to a webpage where people can learn more about the work of this designer/typefoundry.

[code: profile-not-found]
⚠️ WARN Check for codepoints not covered by METADATA subsets.
* ⚠️ **WARN**

The following codepoints supported by the font are not covered by any subsets defined in the font's metadata file, and will never be served. You can solve this by either manually adding additional subset declarations to METADATA.pb, or by editing the glyphset definitions.

  • U+02C7 CARON: try adding one of: yi, tifinagh, canadian-aboriginal
  • U+02D8 BREVE: try adding one of: yi, canadian-aboriginal
  • U+02D9 DOT ABOVE: try adding one of: yi, canadian-aboriginal
  • U+02DB OGONEK: try adding one of: yi, canadian-aboriginal
  • U+02DD DOUBLE ACUTE ACCENT: not included in any glyphset definition
  • U+0302 COMBINING CIRCUMFLEX ACCENT: try adding one of: coptic, tifinagh, cherokee, math
  • U+0306 COMBINING BREVE: try adding one of: tifinagh, old-permic
  • U+0307 COMBINING DOT ABOVE: try adding one of: coptic, tifinagh, syriac, canadian-aboriginal, old-permic, math, tai-le, malayalam
  • U+030A COMBINING RING ABOVE: try adding syriac
  • U+030B COMBINING DOUBLE ACUTE ACCENT: try adding one of: osage, cherokee 55 more.

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: arabic, latin, latin-ext, menu

[code: unreachable-subsetting]
⚠️ WARN Shapes languages in all GF glyphsets.
* ⚠️ **WARN**

GF_Arabic_Plus glyphset:

Language WARN messages
ms_Arab (Malay (Arabic)) No exemplar glyphs were defined for language Malay (Arabic)
[code: warning-language-shaping]
⚠️ WARN Is there kerning info for non-ligated sequences?
* ⚠️ **WARN**

GPOS table lacks kerning info for the following non-ligated sequences:

- uni0644.init + uni0644.medi

- uni0644.medi + uni0651

- uni0651 + uni0670

- uni066E.init + uni062D.fina

- uni066E.init + uni062D.medi

- uni0670 + uni0647.fina

- uni0670 + uni06C1.fina
[code: lacks-kern-info]
ℹ️ INFO List all superfamily filepaths
* ℹ️ **INFO**

ofl/fustat

[code: family-path]
ℹ️ INFO Font contains all required tables?
* ℹ️ **INFO**

This font contains the following optional tables:

- loca

- prep

- GPOS

- GSUB

- gasp
[code: optional-tables]
ℹ️ INFO Does DESCRIPTION file contain a upstream Git repo URL?
* ℹ️ **INFO**

Found a git repo URL: https://github.com/Kief-Type-Foundry/Fustat

[code: url-found]
ℹ️ INFO Check for presence of an ARTICLE.en_us.html file
* ℹ️ **INFO**

This font doesn't have an ARTICLE.en_us.html file.

[code: missing-article]
ℹ️ INFO EPAR table present in font?
* ℹ️ **INFO**

EPAR table not present in font. To learn more see https://github.com/fonttools/fontbakery/issues/818

[code: lacks-EPAR]
ℹ️ INFO Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering?
* ℹ️ **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 Show hinting filesize impact.
* ℹ️ **INFO**

Hinting filesize impact:

ofl/fustat/Fustat[wght].ttf
Dehinted Size 177.4kb
Hinted Size 177.4kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
ℹ️ INFO Ensure fonts have ScriptLangTags declared on the 'meta' table.
* ℹ️ **INFO**

Arab,Latn

[code: dlng-tag] * ℹ️ **INFO**

Arab,Latn

[code: slng-tag]
ℹ️ INFO Font has old ttfautohint applied?
* ℹ️ **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 1.007']

[code: version-not-detected]
[1] Family checks
ℹ️ INFO Check axis ordering on the STAT table.
* ℹ️ **INFO**

None of the fonts lack a STAT table.

And these are the most common STAT axis orderings:
('wght', 1)
[code: summary]

Summary

💥 ERROR ☠ FATAL 🔥 FAIL ⚠️ WARN ⏩ SKIP ℹ️ INFO ✅ PASS 🔎 DEBUG
0 0 0 8 41 11 188 0
0% 0% 0% 3% 17% 4% 76% 0%

Note: The following loglevels were omitted in this report:

khaledhosny commented 2 weeks ago
  • while Khaled engineered

While the order doesn't matter much, just to make things clear: I was also involved in the Arabic design both directly by drawing some glyphs and indirectly by giving feedback to Gaber during the design process.

yanone commented 2 weeks ago

I drew my inspiration for this change from the font's README which is very clear on who did what. Thank you for the clarification.

khaledhosny commented 2 weeks ago

I missed that. Fair enough.