Montserrat: Version 9.000 added #8281
Conversation
FontBakery reportfontbakery version: 0.13.0a1 Check results[24] Montserrat-Italic[wght].ttf🔥 FAIL Checking OS/2 usWinAscent & usWinDescent.
🔥 FAIL Ensure smart dropout control is enabled in "prep" table instructions.
🔥 FAIL Shapes languages in all GF glyphsets.
|
| ofl/montserrat/Montserrat-Italic[wght].ttf | |
|---|---|
| Dehinted Size | 739.5kb |
| Hinted Size | 744.0kb |
| Increase | 4.5kb |
| Change | 0.6 % |
ℹ️ INFO Font contains all required tables?
- ℹ️ INFO
This font contains the following optional tables:
- cvt
- fpgm
- loca
- prep
- GPOS
- GSUB
- gasp
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field.
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Does DESCRIPTION file contain a upstream Git repo URL?
- ℹ️ INFO
Found a git repo URL: https://github.com/JulietaUla/Montserrat
[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 When missing EPAR table, just emit INFO alert fonttools/fontbakery#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][23] Montserrat[wght].ttf
🔥 FAIL Checking OS/2 usWinAscent & usWinDescent.
- 🔥 FAIL
OS/2.usWinDescent value should be equal or greater than 453, but got 370 instead
[code: descent]
🔥 FAIL Ensure smart dropout control is enabled in "prep" table instructions.
- 🔥 FAIL
The 'prep' table does not contain TrueType instructions enabling smart dropout control. To fix, export the font with autohinting enabled, or run ttfautohint on the font, or run the
gftools fix-nonhintingscript.
[code: lacks-smart-dropout]
🔥 FAIL Shapes languages in all GF glyphsets.
- 🔥 FAIL
GF_Phonetics_SinoExt glyphset:
| FAIL messages | Languages |
|---|---|
| The locl feature did not affect Eng | kdh_Latn (Tem), gaa_Latn (Ga), eto_Latn (Eton, Cameroon), nus_Latn (Nuer), sef_Latn (Cebaara Senoufo), bud_Latn (Ntcham), bib_Latn (Bissa), bza_Latn (Bandi), ekm_Latn (Elip), ewo_Latn (Ewondo) and 241 more. |
| The locl feature did not affect uni01A9 | gaa_Latn (Ga) |
| The locl feature did not affect uni01B7 | gaa_Latn (Ga) |
| The locl feature did not affect uni0181 | dnj_Latn (Dan) |
| Shaper didn't attach uni035F to t | udu_Latn (Uduk) |
| Shaper didn't attach uni035F to T | udu_Latn (Uduk) |
| Shaper didn't attach uni035F to t.sc | udu_Latn (Uduk) |
⚠️ WARNGF_Phonetics_SinoExt glyphset:
| WARN messages | Languages |
|---|---|
| No exemplar glyphs were defined for language Amo | amo_Latn (Amo) |
| No exemplar glyphs were defined for language Atsam | cch_Latn (Atsam) |
| No exemplar glyphs were defined for language Eastern Gurung, Latin | ggn_Latn (Eastern Gurung, Latin) |
| No exemplar glyphs were defined for language Koro | kfo_Latn (Koro) |
| No exemplar glyphs were defined for language Mbunga | mgy_Latn (Mbunga) |
| No exemplar glyphs were defined for language Mina | hna_Latn (Mina) |
| No exemplar glyphs were defined for language Seki | syi_Latn (Seki) |
⚠️ WARN Check glyphs in mark glyph class are non-spacing.
⚠️ WARNThe following glyphs seem to be spacing (because they have width > 0 on the hmtx table) so they may be in the GDEF mark glyph class by mistake, or they should have zero width instead:
descendercy (unencoded), descendercy.case (unencoded), descendercy.case.short (unencoded) and descendercy.short (unencoded)
[code: spacing-mark-glyphs]
⚠️ WARN Detect any interpolation issues in the font.
⚠️ WARNInterpolation issues were found in the font:
- Contour 2 start point differs in glyph 'uni0247.ss01.BRACKET.varAlt01' between location wght=100 and location wght=449
- Contour 2 in glyph 'uni0247.ss01.BRACKET.varAlt01': becomes underweight between wght=100 and wght=449.
- Contour 2 start point differs in glyph 'uni20B2.BRACKET.varAlt01' between location wght=100 and location wght=449
- Contour 2 in glyph 'uni20B2.BRACKET.varAlt01': becomes underweight between wght=100 and wght=449.
- Contour order differs in glyph 'uni2C66.sc.ss01.BRACKET.varAlt01': [0, 1, 2, 3] in wght=100, [3, 1, 2, 0] in wght=449.
- Contour order differs in glyph 'uni2C66.sc.ss01.BRACKET.varAlt01': [0, 1, 2, 3] in wght=449, [3, 1, 2, 0] in wght=900.
[code: interpolation-issues]
⚠️ WARN Is there kerning info for non-ligated sequences?
⚠️ WARNGPOS table lacks kerning info for the following non-ligated sequences:
- f + i
- f + l
[code: lacks-kern-info]
⚠️ WARN Check font contains no unreachable glyphs
⚠️ WARNThe following glyphs could not be reached by codepoint or substitution rules:
- Aogonekacute.loclNAV
- Arighthalfring
- Dhookandtail
- Eogonekacute.loclNAV
- I.uc
- Jdotlessstroke
- Napostrophe.ss01
- Oogonekacute.loclNAV
- Tdieresis
- Uogonekacute.loclNAV
- 30 more.
Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]⚠️ WARN Glyph names are all valid?
⚠️ WARNThe following glyph names may be too long for some legacy systems which may expect a maximum 31-characters length limit:
uni2C65.sc.ss01.BRACKET.varAlt01 and uni2C66.sc.ss01.BRACKET.varAlt01
[code: legacy-long-names]
⚠️ WARN Validate size, and resolution of article images, and ensure article page has minimum length and includes visual assets.
⚠️ WARNFamily metadata at ofl/montserrat does not have an article.
[code: lacks-article]
⚠️ WARN Check for codepoints not covered by METADATA subsets.
⚠️ WARNThe 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+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+0302 COMBINING CIRCUMFLEX ACCENT: try adding one of: cherokee, math, tifinagh, coptic
- U+0306 COMBINING BREVE: try adding one of: old-permic, tifinagh
- U+0307 COMBINING DOT ABOVE: try adding one of: math, malayalam, duployan, tai-le, hebrew, tifinagh, todhri, syriac, old-permic, coptic, canadian-aboriginal
- U+030A COMBINING RING ABOVE: try adding one of: duployan, syriac
- U+030B COMBINING DOUBLE ACUTE ACCENT: try adding one of: cherokee, osage
- U+030C COMBINING CARON: try adding one of: cherokee, tai-le
- U+030D COMBINING VERTICAL LINE ABOVE: try adding sunuwar 133 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: cyrillic, cyrillic-ext, latin, latin-ext, menu, vietnamese
⚠️ WARN Ensure soft_dotted characters lose their dot when combined with marks that replace the dot.
⚠️ WARNThe dot of soft dotted characters used in orthographies must disappear in the following strings: і́
The dot of soft dotted characters should disappear in other cases, for example: ɉ̀ ɉ́ ɉ̂ ɉ̃ ɉ̄ ɉ̆ ɉ̇ ɉ̈ ɉ̉ ɉ̊ ɉ̋ ɉ̌ ɉ̍ ɉ̏ ɉ̐ ɉ̑ ɉ̒ ɉ̓ ɉ᷄ ɉ᷅
Your font fully covers the following languages that require the soft-dotted feature: Belarusian (Cyrl, 10,064,517 speakers), Zapotec (Latn, 490,000 speakers), Sar (Latn, 500,000 speakers), Ukrainian (Cyrl, 29,273,587 speakers), Ijo, Southeast (Latn, 2,471,000 speakers), Cicipu (Latn, 44,000 speakers), Dutch (Latn, 31,709,104 speakers), Nzakara (Latn, 50,000 speakers), South Central Banda (Latn, 244,000 speakers), Mango (Latn, 77,000 speakers), Gulay (Latn, 250,478 speakers), Bete-Bendi (Latn, 100,000 speakers), Nateni (Latn, 100,000 speakers), Ebira (Latn, 2,200,000 speakers), Lithuanian (Latn, 2,357,094 speakers), Yala (Latn, 200,000 speakers), Ngbaka (Latn, 1,020,000 speakers), Ekpeye (Latn, 226,000 speakers), Ma’di (Latn, 584,000 speakers), Navajo (Latn, 166,319 speakers).
Your font does not cover the following languages that require the soft-dotted feature: Fur (Latn, 1,230,163 speakers), Ejagham (Latn, 120,000 speakers), Makaa (Latn, 221,000 speakers), Aghem (Latn, 38,843 speakers), Avokaya (Latn, 100,000 speakers), Lugbara (Latn, 2,200,000 speakers), Southern Kisi (Latn, 360,000 speakers), Vute (Latn, 21,000 speakers), Mundani (Latn, 34,000 speakers), Dan (Latn, 1,099,244 speakers), Basaa (Latn, 332,940 speakers), Mfumte (Latn, 79,000 speakers), Dii (Latn, 71,000 speakers), Igbo (Latn, 27,823,640 speakers), Kpelle, Guinea (Latn, 622,000 speakers), Bafut (Latn, 158,146 speakers), Koonzime (Latn, 40,000 speakers), Kom (Latn, 360,685 speakers).
[code: soft-dotted]⚠️ WARN Check the direction of the outermost contour in each glyph
⚠️ WARNThe following glyphs have a counter-clockwise outer contour:
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* 5 more.
Use -F or --full-lists to disable shortening of long lists.
[code: ccw-outer-contour]⚠️ WARN Ensure fonts have ScriptLangTags declared on the 'meta' table.
⚠️ WARNThis font file does not have a 'meta' table.
[code: lacks-meta-table]
ℹ️ INFO Familyname must be unique according to namecheck.fontdata.com
- ℹ️ INFO
The family name "Montserrat" seems to be already in use.
Please visit http://namecheck.fontdata.com for more info.
[code: name-collision]
ℹ️ INFO Show hinting filesize impact.
- ℹ️ INFO
Hinting filesize impact:
| ofl/montserrat/Montserrat[wght].ttf | |
|---|---|
| Dehinted Size | 722.7kb |
| Hinted Size | 727.3kb |
| Increase | 4.5kb |
| Change | 0.6 % |
ℹ️ INFO Font contains all required tables?
- ℹ️ INFO
This font contains the following optional tables:
- cvt
- fpgm
- loca
- prep
- GPOS
- GSUB
- gasp
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field.
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Does DESCRIPTION file contain a upstream Git repo URL?
- ℹ️ INFO
Found a git repo URL: https://github.com/JulietaUla/Montserrat
[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 When missing EPAR table, just emit INFO alert fonttools/fontbakery#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][1] Family checks
Summary
| 💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
|---|---|---|---|---|---|---|---|
| 0 | 0 | 6 | 21 | 73 | 21 | 367 | 0 |
| 0% | 0% | 1% | 4% | 15% | 4% | 75% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
|
Updated |
m4rc1e
force-pushed
the
gftools_packager_ofl_montserrat
branch
from
bfaf290 to
0f7b8a5
Compare
FontBakery reportfontbakery version: 0.13.0a1 Check results[22] Montserrat-Italic[wght].ttf🔥 FAIL Shapes languages in all GF glyphsets.
|
| ofl/montserrat/Montserrat-Italic[wght].ttf | |
|---|---|
| Dehinted Size | 739.5kb |
| Hinted Size | 739.5kb |
| Increase | 24 bytes |
| Change | 0.0 % |
ℹ️ INFO Font contains all required tables?
- ℹ️ INFO
This font contains the following optional tables:
- loca
- prep
- GPOS
- GSUB
- gasp
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field.
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Does DESCRIPTION file contain a upstream Git repo URL?
- ℹ️ INFO
Found a git repo URL: https://github.com/JulietaUla/Montserrat
[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 When missing EPAR table, just emit INFO alert fonttools/fontbakery#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][21] Montserrat[wght].ttf
🔥 FAIL Shapes languages in all GF glyphsets.
- 🔥 FAIL
GF_Phonetics_SinoExt glyphset:
| FAIL messages | Languages |
|---|---|
| The locl feature did not affect Eng | kdh_Latn (Tem), gaa_Latn (Ga), eto_Latn (Eton, Cameroon), nus_Latn (Nuer), sef_Latn (Cebaara Senoufo), bud_Latn (Ntcham), bib_Latn (Bissa), bza_Latn (Bandi), ekm_Latn (Elip), ewo_Latn (Ewondo) and 241 more. |
| The locl feature did not affect uni01A9 | gaa_Latn (Ga) |
| The locl feature did not affect uni01B7 | gaa_Latn (Ga) |
| The locl feature did not affect uni0181 | dnj_Latn (Dan) |
| Shaper didn't attach uni035F to t | udu_Latn (Uduk) |
| Shaper didn't attach uni035F to T | udu_Latn (Uduk) |
| Shaper didn't attach uni035F to t.sc | udu_Latn (Uduk) |
⚠️ WARNGF_Phonetics_SinoExt glyphset:
| WARN messages | Languages |
|---|---|
| No exemplar glyphs were defined for language Amo | amo_Latn (Amo) |
| No exemplar glyphs were defined for language Atsam | cch_Latn (Atsam) |
| No exemplar glyphs were defined for language Eastern Gurung, Latin | ggn_Latn (Eastern Gurung, Latin) |
| No exemplar glyphs were defined for language Koro | kfo_Latn (Koro) |
| No exemplar glyphs were defined for language Mbunga | mgy_Latn (Mbunga) |
| No exemplar glyphs were defined for language Mina | hna_Latn (Mina) |
| No exemplar glyphs were defined for language Seki | syi_Latn (Seki) |
⚠️ WARN Check glyphs in mark glyph class are non-spacing.
⚠️ WARNThe following glyphs seem to be spacing (because they have width > 0 on the hmtx table) so they may be in the GDEF mark glyph class by mistake, or they should have zero width instead:
descendercy (unencoded), descendercy.case (unencoded), descendercy.case.short (unencoded) and descendercy.short (unencoded)
[code: spacing-mark-glyphs]
⚠️ WARN Detect any interpolation issues in the font.
⚠️ WARNInterpolation issues were found in the font:
- Contour 2 start point differs in glyph 'uni20B2.BRACKET.varAlt01' between location wght=100 and location wght=449
- Contour 2 in glyph 'uni20B2.BRACKET.varAlt01': becomes underweight between wght=100 and wght=449.
- Contour 2 start point differs in glyph 'uni0247.ss01.BRACKET.varAlt01' between location wght=100 and location wght=449
- Contour 2 in glyph 'uni0247.ss01.BRACKET.varAlt01': becomes underweight between wght=100 and wght=449.
- Contour order differs in glyph 'uni2C66.sc.ss01.BRACKET.varAlt01': [0, 1, 2, 3] in wght=100, [3, 1, 2, 0] in wght=449.
- Contour order differs in glyph 'uni2C66.sc.ss01.BRACKET.varAlt01': [0, 1, 2, 3] in wght=449, [3, 1, 2, 0] in wght=900.
[code: interpolation-issues]
⚠️ WARN Is there kerning info for non-ligated sequences?
⚠️ WARNGPOS table lacks kerning info for the following non-ligated sequences:
- f + i
- f + l
[code: lacks-kern-info]
⚠️ WARN Check font contains no unreachable glyphs
⚠️ WARNThe following glyphs could not be reached by codepoint or substitution rules:
- Aogonekacute.loclNAV
- Arighthalfring
- Dhookandtail
- Eogonekacute.loclNAV
- I.uc
- Jdotlessstroke
- Napostrophe.ss01
- Oogonekacute.loclNAV
- Tdieresis
- Uogonekacute.loclNAV
- 30 more.
Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]⚠️ WARN Glyph names are all valid?
⚠️ WARNThe following glyph names may be too long for some legacy systems which may expect a maximum 31-characters length limit:
uni2C65.sc.ss01.BRACKET.varAlt01 and uni2C66.sc.ss01.BRACKET.varAlt01
[code: legacy-long-names]
⚠️ WARN Validate size, and resolution of article images, and ensure article page has minimum length and includes visual assets.
⚠️ WARNFamily metadata at ofl/montserrat does not have an article.
[code: lacks-article]
⚠️ WARN Check for codepoints not covered by METADATA subsets.
⚠️ WARNThe 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+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+0302 COMBINING CIRCUMFLEX ACCENT: try adding one of: coptic, tifinagh, math, cherokee
- U+0306 COMBINING BREVE: try adding one of: tifinagh, old-permic
- U+0307 COMBINING DOT ABOVE: try adding one of: math, malayalam, tifinagh, canadian-aboriginal, todhri, coptic, old-permic, tai-le, syriac, duployan, hebrew
- U+030A COMBINING RING ABOVE: try adding one of: syriac, duployan
- U+030B COMBINING DOUBLE ACUTE ACCENT: try adding one of: osage, cherokee
- U+030C COMBINING CARON: try adding one of: tai-le, cherokee
- U+030D COMBINING VERTICAL LINE ABOVE: try adding sunuwar 133 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: cyrillic, cyrillic-ext, latin, latin-ext, menu, vietnamese
⚠️ WARN Ensure soft_dotted characters lose their dot when combined with marks that replace the dot.
⚠️ WARNThe dot of soft dotted characters used in orthographies must disappear in the following strings: і́
The dot of soft dotted characters should disappear in other cases, for example: ɉ̀ ɉ́ ɉ̂ ɉ̃ ɉ̄ ɉ̆ ɉ̇ ɉ̈ ɉ̉ ɉ̊ ɉ̋ ɉ̌ ɉ̍ ɉ̏ ɉ̐ ɉ̑ ɉ̒ ɉ̓ ɉ᷄ ɉ᷅
Your font fully covers the following languages that require the soft-dotted feature: Ekpeye (Latn, 226,000 speakers), Dutch (Latn, 31,709,104 speakers), Nzakara (Latn, 50,000 speakers), Lithuanian (Latn, 2,357,094 speakers), Sar (Latn, 500,000 speakers), Cicipu (Latn, 44,000 speakers), South Central Banda (Latn, 244,000 speakers), Ukrainian (Cyrl, 29,273,587 speakers), Mango (Latn, 77,000 speakers), Navajo (Latn, 166,319 speakers), Zapotec (Latn, 490,000 speakers), Ma’di (Latn, 584,000 speakers), Yala (Latn, 200,000 speakers), Ijo, Southeast (Latn, 2,471,000 speakers), Belarusian (Cyrl, 10,064,517 speakers), Nateni (Latn, 100,000 speakers), Bete-Bendi (Latn, 100,000 speakers), Ngbaka (Latn, 1,020,000 speakers), Ebira (Latn, 2,200,000 speakers), Gulay (Latn, 250,478 speakers).
Your font does not cover the following languages that require the soft-dotted feature: Kom (Latn, 360,685 speakers), Mundani (Latn, 34,000 speakers), Igbo (Latn, 27,823,640 speakers), Makaa (Latn, 221,000 speakers), Mfumte (Latn, 79,000 speakers), Avokaya (Latn, 100,000 speakers), Lugbara (Latn, 2,200,000 speakers), Kpelle, Guinea (Latn, 622,000 speakers), Aghem (Latn, 38,843 speakers), Basaa (Latn, 332,940 speakers), Dan (Latn, 1,099,244 speakers), Koonzime (Latn, 40,000 speakers), Vute (Latn, 21,000 speakers), Southern Kisi (Latn, 360,000 speakers), Dii (Latn, 71,000 speakers), Bafut (Latn, 158,146 speakers), Fur (Latn, 1,230,163 speakers), Ejagham (Latn, 120,000 speakers).
[code: soft-dotted]⚠️ WARN Check the direction of the outermost contour in each glyph
⚠️ WARNThe following glyphs have a counter-clockwise outer contour:
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* 5 more.
Use -F or --full-lists to disable shortening of long lists.
[code: ccw-outer-contour]⚠️ WARN Ensure fonts have ScriptLangTags declared on the 'meta' table.
⚠️ WARNThis font file does not have a 'meta' table.
[code: lacks-meta-table]
ℹ️ INFO Familyname must be unique according to namecheck.fontdata.com
- ℹ️ INFO
The family name "Montserrat" seems to be already in use.
Please visit http://namecheck.fontdata.com for more info.
[code: name-collision]
ℹ️ INFO Show hinting filesize impact.
- ℹ️ INFO
Hinting filesize impact:
| ofl/montserrat/Montserrat[wght].ttf | |
|---|---|
| Dehinted Size | 722.7kb |
| Hinted Size | 722.8kb |
| Increase | 24 bytes |
| Change | 0.0 % |
ℹ️ INFO Font contains all required tables?
- ℹ️ INFO
This font contains the following optional tables:
- loca
- prep
- GPOS
- GSUB
- gasp
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field.
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Does DESCRIPTION file contain a upstream Git repo URL?
- ℹ️ INFO
Found a git repo URL: https://github.com/JulietaUla/Montserrat
[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 When missing EPAR table, just emit INFO alert fonttools/fontbakery#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][1] Family checks
Summary
| 💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
|---|---|---|---|---|---|---|---|
| 0 | 0 | 2 | 21 | 75 | 21 | 369 | 0 |
| 0% | 0% | 0% | 4% | 15% | 4% | 76% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
|
Hey @m4rc1e, here you can find my review! Fontbakery:
Software-> Menus, rendering, kerning, features.
Screen.Recording.2024-10-09.at.14.03.49.movHinting
Screen.Recording.2024-10-09.at.14.21.13.movScreen.Recording.2024-10-09.at.14.21.56.movBracket layers
Screen.Recording.2024-10-09.at.13.59.05.mov
Anchors placement
Screen.Recording.2024-10-09.at.14.30.13.movOther bugs
Screen.Recording.2024-10-09.at.14.30.40.movScreen.Recording.2024-10-09.at.14.35.20.mov |
|
Brilliant review thank you very much! @juandelperal I'll take a look at what could be causing the Indesign issue. Mind looking at the rest? |
m4rc1e
added
P2
-- Upstream is working on it
|
Hi Emma,
Yes, we designed it to switch the position like that, otherwise the inner counter got covered. Other bugs Screen.Recording.2024-10-16.at.10.33.34.movScreen.Recording.2024-10-16.at.10.34.58.movIndesign |
|
Hi @juandelperal, thank you for the update! Bug with I saw the problem in Fontgauntlet, and I have it also in Font Goggles and Indesign: 
Indesign: Good news! The issue seems to be gone with the Indesign upgrade (2025 version LATIN CAPITAL LETTER T WITH DIAGONAL STROKE Some alternate have been drawn for the bolder versions, but they still don't work. 
For the default version Screen.Recording.2024-10-16.at.12.04.22.mov |
|
There is also a random bug with the statics, variable version looks fine |
|
Thanks Emma It seems to be a problem with gftools builder. Here the comparison gftools.movglyphs.movWe also attach the font generated from glyphs to check |
|
Ok I see! Well, I don't think it should be a blocker, @m4rc1e what do you thing? https://github.com/googlefonts/glyphsLib/issues?q=is%3Aissue%20state%3Aopen%20layer |
|
Hi Emma and Marc! |
Taken from the upstream repo https://github.com/JulietaUla/Montserrat at commit JulietaUla/Montserrat@f025b64.
|
Updated |
m4rc1e
force-pushed
the
gftools_packager_ofl_montserrat
branch
from
0f7b8a5 to
76fca9f
Compare
FontBakery reportfontbakery version: 0.13.0a2 Check results[24] Montserrat-Italic[wght].ttf🔥 FAIL Check there are no overlapping path segments
[code: overlapping-path-segments] 🔥 FAIL Ensure smart dropout control is enabled in "prep" table instructions.
🔥 FAIL Shapes languages in all GF glyphsets.
|
| ofl/montserrat/Montserrat-Italic[wght].ttf | |
|---|---|
| Dehinted Size | 740.0kb |
| Hinted Size | 744.5kb |
| Increase | 4.5kb |
| Change | 0.6 % |
ℹ️ INFO Font contains all required tables?
- ℹ️ INFO
This font contains the following optional tables:
- cvt
- fpgm
- loca
- prep
- GPOS
- GSUB
- gasp
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field.
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Does DESCRIPTION file contain a upstream Git repo URL?
- ℹ️ INFO
Found a git repo URL: https://github.com/JulietaUla/Montserrat
[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 When missing EPAR table, just emit INFO alert fonttools/fontbakery#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][23] Montserrat[wght].ttf
🔥 FAIL Check there are no overlapping path segments
- 🔥 FAIL
The following glyphs have overlapping path segments:
* uni1E28 (U+1E28): L<<153.0,0.0>--<133.0,0.0>> has the same coordinates as a previous segment.
* Iogonek (U+012E): L<<133.0,0.0>--<153.0,0.0>> has the same coordinates as a previous segment.
* uni1E29 (U+1E29): L<<115.0,0.0>--<135.0,0.0>> has the same coordinates as a previous segment.
* uni01310328: L<<115.0,0.0>--<135.0,0.0>> has the same coordinates as a previous segment.
* iogonek (U+012F): L<<115.0,0.0>--<135.0,0.0>> has the same coordinates as a previous segment.
* iogonek.dotless: L<<115.0,0.0>--<135.0,0.0>> has the same coordinates as a previous segment.
* iogonek.sc: L<<122.0,0.0>--<142.0,0.0>> has the same coordinates as a previous segment.
* uni040A (U+040A): L<<649.0,700.0>--<669.0,700.0>> has the same coordinates as a previous segment.
* uni04A4 (U+04A4): L<<667.0,0.0>--<667.0,700.0>> has the same coordinates as a previous segment.
* uni04A4 (U+04A4): L<<687.0,0.0>--<667.0,0.0>> has the same coordinates as a previous segment.
* uni04A5 (U+04A5): L<<546.0,0.0>--<526.0,0.0>> has the same coordinates as a previous segment.
* uni20AD (U+20AD): L<<171.0,350.0>--<199.0,350.0>> has the same coordinates as a previous segment.
[code: overlapping-path-segments]
🔥 FAIL Ensure smart dropout control is enabled in "prep" table instructions.
- 🔥 FAIL
The 'prep' table does not contain TrueType instructions enabling smart dropout control. To fix, export the font with autohinting enabled, or run ttfautohint on the font, or run the
gftools fix-nonhintingscript.
[code: lacks-smart-dropout]
🔥 FAIL Shapes languages in all GF glyphsets.
- 🔥 FAIL
GF_Phonetics_SinoExt glyphset:
| FAIL messages | Languages |
|---|---|
| The locl feature did not affect Eng | bkm_Latn (Kom), tnr_Latn (Ménik), gde_Latn (Gude), bsc_Latn (Bassari), ee_Latn (Ewe), ken_Latn (Kenyang), muy_Latn (Muyang), kao_Latn (Xaasongaxango), dur_Latn (Dii), kqs_Latn (Kissi, Northern) and 245 more. |
| Shaper didn't attach uni035F to T | udu_Latn (Uduk) |
| Shaper didn't attach uni035F to t | udu_Latn (Uduk) |
| Shaper didn't attach uni035F to t.sc | udu_Latn (Uduk) |
| The locl feature did not affect uni01A9 | gaa_Latn (Ga) |
| The locl feature did not affect uni01B7 | gaa_Latn (Ga) |
| The locl feature did not affect uni0181 | dnj_Latn (Dan) |
⚠️ WARNGF_Phonetics_SinoExt glyphset:
| WARN messages | Languages |
|---|---|
| No exemplar glyphs were defined for language Eastern Gurung, Latin | ggn_Latn (Eastern Gurung, Latin) |
| No exemplar glyphs were defined for language Mina | hna_Latn (Mina) |
| No exemplar glyphs were defined for language Seki | syi_Latn (Seki) |
| No exemplar glyphs were defined for language Atsam | cch_Latn (Atsam) |
| No exemplar glyphs were defined for language Koro | kfo_Latn (Koro) |
| No exemplar glyphs were defined for language Mbunga | mgy_Latn (Mbunga) |
| No exemplar glyphs were defined for language Amo | amo_Latn (Amo) |
⚠️ WARN Check glyphs in mark glyph class are non-spacing.
⚠️ WARNThe following glyphs seem to be spacing (because they have width > 0 on the hmtx table) so they may be in the GDEF mark glyph class by mistake, or they should have zero width instead:
descendercy (unencoded), descendercy.case (unencoded), descendercy.case.short (unencoded) and descendercy.short (unencoded)
[code: spacing-mark-glyphs]
⚠️ WARN Detect any interpolation issues in the font.
⚠️ WARNInterpolation issues were found in the font:
- Contour 2 start point differs in glyph 'uni0247.ss01.BRACKET.varAlt01' between location wght=100 and location wght=449
- Contour 2 in glyph 'uni0247.ss01.BRACKET.varAlt01': becomes underweight between wght=100 and wght=449.
- Contour 2 start point differs in glyph 'uni20B2.BRACKET.varAlt01' between location wght=100 and location wght=449
- Contour 2 in glyph 'uni20B2.BRACKET.varAlt01': becomes underweight between wght=100 and wght=449.
[code: interpolation-issues]
⚠️ WARN Is there kerning info for non-ligated sequences?
⚠️ WARNGPOS table lacks kerning info for the following non-ligated sequences:
- f + i
- f + l
[code: lacks-kern-info]
⚠️ WARN Check font contains no unreachable glyphs
⚠️ WARNThe following glyphs could not be reached by codepoint or substitution rules:
- Aogonekacute.loclNAV
- Arighthalfring
- Dhookandtail
- Eogonekacute.loclNAV
- I.uc
- Jdotlessstroke
- Napostrophe.ss01
- Oogonekacute.loclNAV
- Tdieresis
- Uogonekacute.loclNAV
- 30 more.
Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]⚠️ WARN Glyph names are all valid?
⚠️ WARNThe following glyph names may be too long for some legacy systems which may expect a maximum 31-characters length limit:
uni2C65.sc.ss01.BRACKET.varAlt01 and uni2C66.sc.ss01.BRACKET.varAlt01
[code: legacy-long-names]
⚠️ WARN Validate size, and resolution of article images, and ensure article page has minimum length and includes visual assets.
⚠️ WARNFamily metadata at ofl/montserrat does not have an article.
[code: lacks-article]
⚠️ WARN Check for codepoints not covered by METADATA subsets.
⚠️ WARNThe 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+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+0302 COMBINING CIRCUMFLEX ACCENT: try adding one of: tifinagh, math, coptic, cherokee
- U+0306 COMBINING BREVE: try adding one of: tifinagh, old-permic
- U+0307 COMBINING DOT ABOVE: try adding one of: malayalam, tai-le, coptic, duployan, old-permic, canadian-aboriginal, syriac, hebrew, tifinagh, todhri, math
- U+030A COMBINING RING ABOVE: try adding one of: syriac, duployan
- U+030B COMBINING DOUBLE ACUTE ACCENT: try adding one of: osage, cherokee
- U+030C COMBINING CARON: try adding one of: tai-le, cherokee
- U+030D COMBINING VERTICAL LINE ABOVE: try adding sunuwar 133 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: cyrillic, cyrillic-ext, latin, latin-ext, menu, vietnamese
⚠️ WARN Ensure soft_dotted characters lose their dot when combined with marks that replace the dot.
⚠️ WARNThe dot of soft dotted characters used in orthographies must disappear in the following strings: і́
The dot of soft dotted characters should disappear in other cases, for example: ɉ̀ ɉ́ ɉ̂ ɉ̃ ɉ̄ ɉ̆ ɉ̇ ɉ̈ ɉ̉ ɉ̊ ɉ̋ ɉ̌ ɉ̍ ɉ̏ ɉ̐ ɉ̑ ɉ̒ ɉ̓ ɉ᷄ ɉ᷅
Your font fully covers the following languages that require the soft-dotted feature: Ekpeye (Latn, 226,000 speakers), Ngbaka (Latn, 1,020,000 speakers), Bete-Bendi (Latn, 100,000 speakers), Kaska (Latn, 125 speakers), Sar (Latn, 500,000 speakers), Yala (Latn, 200,000 speakers), Nzakara (Latn, 50,000 speakers), Gulay (Latn, 250,478 speakers), Dutch (Latn, 31,709,104 speakers), Mango (Latn, 77,000 speakers), Cicipu (Latn, 44,000 speakers), Navajo (Latn, 166,319 speakers), Belarusian (Cyrl, 10,064,517 speakers), Heiltsuk (Latn, 300 speakers), South Central Banda (Latn, 244,000 speakers), Ebira (Latn, 2,200,000 speakers), Ijo, Southeast (Latn, 2,471,000 speakers), Ma’di (Latn, 584,000 speakers), Lithuanian (Latn, 2,357,094 speakers), Han (Latn, 6 speakers), Zapotec (Latn, 490,000 speakers), Nateni (Latn, 100,000 speakers), Ukrainian (Cyrl, 29,273,587 speakers).
Your font does not cover the following languages that require the soft-dotted feature: Teke-Ebo (Latn, 260,000 speakers), Lugbara (Latn, 2,200,000 speakers), Koonzime (Latn, 40,000 speakers), Dan (Latn, 1,099,244 speakers), Igbo (Latn, 27,823,640 speakers), Makaa (Latn, 221,000 speakers), Fur (Latn, 1,230,163 speakers), Basaa (Latn, 332,940 speakers), Bafut (Latn, 158,146 speakers), Kom (Latn, 360,685 speakers), Southern Kisi (Latn, 360,000 speakers), Mundani (Latn, 34,000 speakers), Avokaya (Latn, 100,000 speakers), Vute (Latn, 21,000 speakers), Dii (Latn, 71,000 speakers), Ejagham (Latn, 120,000 speakers), Aghem (Latn, 38,843 speakers), Mfumte (Latn, 79,000 speakers), Kpelle, Guinea (Latn, 622,000 speakers).
[code: soft-dotted]⚠️ WARN Check the direction of the outermost contour in each glyph
⚠️ WARNThe following glyphs have a counter-clockwise outer contour:
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* uni25CC (U+25CC) has a counter-clockwise outer contour
* 5 more.
Use -F or --full-lists to disable shortening of long lists.
[code: ccw-outer-contour]⚠️ WARN Ensure fonts have ScriptLangTags declared on the 'meta' table.
⚠️ WARNThis font file does not have a 'meta' table.
[code: lacks-meta-table]
ℹ️ INFO Familyname must be unique according to namecheck.fontdata.com
- ℹ️ INFO
The family name "Montserrat" seems to be already in use.
Please visit http://namecheck.fontdata.com for more info.
[code: name-collision]
ℹ️ INFO Show hinting filesize impact.
- ℹ️ INFO
Hinting filesize impact:
| ofl/montserrat/Montserrat[wght].ttf | |
|---|---|
| Dehinted Size | 723.0kb |
| Hinted Size | 727.5kb |
| Increase | 4.5kb |
| Change | 0.6 % |
ℹ️ INFO Font contains all required tables?
- ℹ️ INFO
This font contains the following optional tables:
- cvt
- fpgm
- loca
- prep
- GPOS
- GSUB
- gasp
[code: optional-tables]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field.
- ℹ️ INFO
Please consider adding a family.minisite_url entry.
[code: lacks-minisite-url]
ℹ️ INFO Does DESCRIPTION file contain a upstream Git repo URL?
- ℹ️ INFO
Found a git repo URL: https://github.com/JulietaUla/Montserrat
[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 When missing EPAR table, just emit INFO alert fonttools/fontbakery#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][1] Family checks
Summary
| 💥 ERROR | ☠ FATAL | 🔥 FAIL | ⏩ SKIP | ℹ️ INFO | ✅ PASS | 🔎 DEBUG | |
|---|---|---|---|---|---|---|---|
| 0 | 0 | 6 | 21 | 73 | 21 | 369 | 0 |
| 0% | 0% | 1% | 4% | 15% | 4% | 75% | 0% |
Note: The following loglevels were omitted in this report:
- SKIP
- PASS
- DEBUG
|
@juandelperal Thank you very much. The pr here now reflects your latest changes. @emmamarichal Feel free to do another review. I don't mind if skip the review and I'll take another look. |
|
@m4rc1e I can do a quick one tomorrow morning! |
|
New fontbakery check:
|
I think this check may already be redundant since Chrome 130 has fixed it.
This shouldn't be enabled since the font is manually hinted. |
emmamarichal
removed
the
-- Upstream is working on it
No description provided.