Skip to content
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

Montserrat: Version 9.000 added #8281

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

m4rc1e
Copy link
Collaborator

@m4rc1e m4rc1e commented Oct 7, 2024

No description provided.

Copy link

github-actions bot commented Oct 7, 2024

FontBakery report

fontbakery version: 0.13.0a1

Check results

[24] Montserrat-Italic[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-nonhinting script.


    [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)
[code: failed-language-shaping]
  • ⚠️ WARN

    GF_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)
[code: warning-language-shaping]
⚠️ WARN Check glyphs in mark glyph class are non-spacing.
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    Interpolation issues were found in the font:

- Contour 2 start point differs in glyph 'uni0247.BRACKET.varAlt01' between location wght=100 and location wght=449

- Contour 2 in glyph 'uni0247.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 'uni023E.ss01.BRACKET.varAlt01': [0, 1, 2, 3] in wght=100, [1, 0, 2, 3] in wght=449.

[code: interpolation-issues]

⚠️ WARN Is there kerning info for non-ligated sequences?
  • ⚠️ WARN

    GPOS 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
  • ⚠️ WARN

    The 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

- 29 more.

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

[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:
    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.
  • ⚠️ WARN

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


    [code: lacks-article]
⚠️ 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+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

[code: unreachable-subsetting]
⚠️ WARN On a family update, the DESCRIPTION.en_us.html file should ideally also be updated.
  • ⚠️ WARN

    The DESCRIPTION.en_us.html file in this family has not changed in comparison to the latest font release on the google/fonts github repo.
    Please consider mentioning note-worthy improvements made to the family recently.


    [code: description-not-updated]
⚠️ WARN Ensure soft_dotted characters lose their dot when combined with marks that replace the dot.
  • ⚠️ WARN

    The 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
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    This font file does not have a 'meta' table.


    [code: lacks-meta-table]
ℹ️ INFO List all superfamily filepaths
  • ℹ️ INFO

    ofl/montserrat


    [code: family-path]
ℹ️ 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-Italic[wght].ttf
Dehinted Size 739.5kb
Hinted Size 744.0kb
Increase 4.5kb
Change 0.6 %
[code: size-impact]
ℹ️ 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 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 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 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 9.000']


    [code: version-not-detected]
[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-nonhinting script.


    [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)
[code: failed-language-shaping]
  • ⚠️ WARN

    GF_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)
[code: warning-language-shaping]
⚠️ WARN Check glyphs in mark glyph class are non-spacing.
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    Interpolation 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?
  • ⚠️ WARN

    GPOS 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
  • ⚠️ WARN

    The 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?
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

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


    [code: lacks-article]
⚠️ 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+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

[code: unreachable-subsetting]
⚠️ WARN Ensure soft_dotted characters lose their dot when combined with marks that replace the dot.
  • ⚠️ WARN

    The 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
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    This font file does not have a 'meta' table.


    [code: lacks-meta-table]
ℹ️ INFO List all superfamily filepaths
  • ℹ️ INFO

    ofl/montserrat


    [code: family-path]
ℹ️ 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 %
[code: size-impact]
ℹ️ 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 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 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 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 9.000']


    [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-ital', 2)

[code: summary]

Summary

💥 ERROR ☠ FATAL 🔥 FAIL ⚠️ WARN ⏩ 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

@m4rc1e
Copy link
Collaborator Author

m4rc1e commented Oct 8, 2024

Updated

@m4rc1e m4rc1e force-pushed the gftools_packager_ofl_montserrat branch from bfaf290 to 0f7b8a5 Compare October 8, 2024 09:41
Copy link

github-actions bot commented Oct 8, 2024

FontBakery report

fontbakery version: 0.13.0a1

Check results

[22] Montserrat-Italic[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)
[code: failed-language-shaping]
  • ⚠️ WARN

    GF_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)
[code: warning-language-shaping]
⚠️ WARN Check glyphs in mark glyph class are non-spacing.
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    Interpolation issues were found in the font:

- Contour order differs in glyph 'uni023E.ss01.BRACKET.varAlt01': [0, 1, 2, 3] in wght=100, [1, 0, 2, 3] in 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 2 start point differs in glyph 'uni0247.BRACKET.varAlt01' between location wght=100 and location wght=449

- Contour 2 in glyph 'uni0247.BRACKET.varAlt01': becomes underweight between wght=100 and wght=449.

[code: interpolation-issues]

⚠️ WARN Is there kerning info for non-ligated sequences?
  • ⚠️ WARN

    GPOS 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
  • ⚠️ WARN

    The 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

- 29 more.

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

[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:
    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.
  • ⚠️ WARN

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


    [code: lacks-article]
⚠️ 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+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

[code: unreachable-subsetting]
⚠️ WARN On a family update, the DESCRIPTION.en_us.html file should ideally also be updated.
  • ⚠️ WARN

    The DESCRIPTION.en_us.html file in this family has not changed in comparison to the latest font release on the google/fonts github repo.
    Please consider mentioning note-worthy improvements made to the family recently.


    [code: description-not-updated]
⚠️ WARN Ensure soft_dotted characters lose their dot when combined with marks that replace the dot.
  • ⚠️ WARN

    The 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
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    This font file does not have a 'meta' table.


    [code: lacks-meta-table]
ℹ️ INFO List all superfamily filepaths
  • ℹ️ INFO

    ofl/montserrat


    [code: family-path]
ℹ️ 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-Italic[wght].ttf
Dehinted Size 739.5kb
Hinted Size 739.5kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
ℹ️ 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 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 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 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 9.000']


    [code: version-not-detected]
[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)
[code: failed-language-shaping]
  • ⚠️ WARN

    GF_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)
[code: warning-language-shaping]
⚠️ WARN Check glyphs in mark glyph class are non-spacing.
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    Interpolation 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?
  • ⚠️ WARN

    GPOS 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
  • ⚠️ WARN

    The 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?
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

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


    [code: lacks-article]
⚠️ 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+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

[code: unreachable-subsetting]
⚠️ WARN Ensure soft_dotted characters lose their dot when combined with marks that replace the dot.
  • ⚠️ WARN

    The 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
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    This font file does not have a 'meta' table.


    [code: lacks-meta-table]
ℹ️ INFO List all superfamily filepaths
  • ℹ️ INFO

    ofl/montserrat


    [code: family-path]
ℹ️ 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 %
[code: size-impact]
ℹ️ 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 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 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 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 9.000']


    [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-ital', 2)

[code: summary]

Summary

💥 ERROR ☠ FATAL 🔥 FAIL ⚠️ WARN ⏩ 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

@emmamarichal emmamarichal self-requested a review October 9, 2024 09:18
@emmamarichal emmamarichal self-assigned this Oct 9, 2024
@emmamarichal
Copy link
Collaborator

Hey @m4rc1e, here you can find my review!

Fontbakery:

⚠️ WARN On a family update, the DESCRIPTION.en_us.html file should ideally also be updated.

  • Maybe we should add a sentence about the update? (SSA related)

Software

-> Menus, rendering, kerning, features.

  • Word Windows: ok
Screenshot 2024-10-09 at 14 15 43
  • TextEdit Mac: ok
  • Word Mac: ok
  • Indesign Mac: not ok
    Slider and style don't show up well:
Screenshot 2024-10-09 at 14 03 02 Screenshot 2024-10-09 at 14 03 16
Screen.Recording.2024-10-09.at.14.03.49.mov

Hinting

  • Here you can find the difference on Windows:
Screen.Recording.2024-10-09.at.14.21.13.mov
Screen.Recording.2024-10-09.at.14.21.56.mov

Bracket layers

  • There are some interpolation issues in the bracket layers (see the generated pdf in the qa), but they don't seem to have any impact on the rendering, so I'll not consider it as a blocker.
Screen.Recording.2024-10-09.at.13.59.05.mov
  • The bracket layer doesn't work for the small cap version (in normal version and in the stylistic set):
Screenshot 2024-10-09 at 14 01 55

Anchors placement

  • The following issues are mostly present in italic too (please check)
Screenshot 2024-10-09 at 14 48 09 Screenshot 2024-10-09 at 14 50 05 Screenshot 2024-10-09 at 14 29 00 Screenshot 2024-10-09 at 14 33 37
  • is this bar placement ok?
Screen.Recording.2024-10-09.at.14.30.13.mov

Other bugs

  • this switch doesn't look right to me:
Screen.Recording.2024-10-09.at.14.30.40.mov
Screen.Recording.2024-10-09.at.14.35.20.mov

@emmamarichal emmamarichal linked an issue Oct 9, 2024 that may be closed by this pull request
@m4rc1e
Copy link
Collaborator Author

m4rc1e commented Oct 9, 2024

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 m4rc1e added P2 -- Upstream is working on it Designer is making changes in the upstream repo and removed P2 labels Oct 14, 2024
@juandelperal
Copy link
Contributor

Hi Emma,
Thanks a lot for this super detailed review! Amazing work.
We fixed all the issues you mentioned, like anchor positioning, hcedilla.sc and change the breakpoint of tdiagonalstroke.sc. You can review them from our latest commits.

is this bar placement ok?

Yes, we designed it to switch the position like that, otherwise the inner counter got covered.

Other bugs
Still, we couldn't reproduce what you mentioned there
We attach some screen recordings from Samsa

Screen.Recording.2024-10-16.at.10.33.34.mov
Screen.Recording.2024-10-16.at.10.34.58.mov

Indesign
Is this problem present in the previous version of the family?
We had so many issues with Indesign support for Variable Fonts. I think something worth trying is dropping the stat table to test if the problem persist. Let us know if we can help @m4rc1e

@emmamarichal
Copy link
Collaborator

Hi @juandelperal, thank you for the update!

Bug with and ɇ / small caps activated:

I saw the problem in Fontgauntlet, and I have it also in Font Goggles and Indesign:
(in bold versions, they should appear as small caps)

Screenshot 2024-10-16 at 11 45 09 Screenshot 2024-10-16 at 11 45 23

Indesign:

Good news! The issue seems to be gone with the Indesign upgrade (2025 version v20.0). cc @m4rc1e

LATIN CAPITAL LETTER T WITH DIAGONAL STROKE

Some alternate have been drawn for the bolder versions, but they still don't work.

Screenshot 2024-10-16 at 11 51 41 Screenshot 2024-10-16 at 11 51 55

For the default version Ⱦ, the alternate works, but when we activate the SS01, with the other T shape, we loose the ss01 shape after 700 (bold).

Screen.Recording.2024-10-16.at.12.04.22.mov

@juandelperal
Copy link
Contributor

There is also a random bug with the statics, variable version looks fine
JulietaUla/Montserrat#150 (comment)

@juandelperal
Copy link
Contributor

Thanks Emma

It seems to be a problem with gftools builder. Here the comparison

gftools.mov
glyphs.mov

We also attach the font generated from glyphs to check
MontserratVF.ttf.zip

@emmamarichal
Copy link
Collaborator

Ok I see! Well, I don't think it should be a blocker, @m4rc1e what do you thing?
GlyphsLib had a lot of issues with alternate layers in the past, I'll re-read the issues to see if a solution exists or if the problem have been raised already.

https://github.com/googlefonts/glyphsLib/issues?q=is%3Aissue%20state%3Aopen%20layer

@m4rc1e m4rc1e added P2 and removed P2 labels Oct 18, 2024
@juandelperal
Copy link
Contributor

Hi Emma and Marc!
We made some fixes. Please update this PR to the last commit before merging.

@m4rc1e
Copy link
Collaborator Author

m4rc1e commented Oct 21, 2024

Updated

@m4rc1e m4rc1e force-pushed the gftools_packager_ofl_montserrat branch from 0f7b8a5 to 76fca9f Compare October 21, 2024 09:33
Copy link

FontBakery report

fontbakery version: 0.13.0a2

Check results

[24] Montserrat-Italic[wght].ttf
🔥 FAIL Check there are no overlapping path segments
  • 🔥 FAIL

    The following glyphs have overlapping path segments:

* uni1E28 (U+1E28): L<<101.0,0.0>--<81.0,0.0>> has the same coordinates as a previous segment.

* uni1E29 (U+1E29): L<<89.0,0.0>--<69.0,0.0>> has the same coordinates as a previous segment.

* uni040A (U+040A): L<<737.0,700.0>--<757.0,700.0>> has the same coordinates as a previous segment.

* uni04A4 (U+04A4): L<<615.0,0.0>--<755.0,700.0>> has the same coordinates as a previous segment.

* uni04A4 (U+04A4): L<<635.0,0.0>--<615.0,0.0>> has the same coordinates as a previous segment.

* uni04A5 (U+04A5): L<<493.0,0.0>--<473.0,0.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-nonhinting script.


    [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)
[code: failed-language-shaping]
  • ⚠️ WARN

    GF_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)
[code: warning-language-shaping]
⚠️ WARN Check glyphs in mark glyph class are non-spacing.
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    Interpolation 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 'uni0247.BRACKET.varAlt01' between location wght=100 and location wght=449

- Contour 2 in glyph 'uni0247.BRACKET.varAlt01': becomes underweight between wght=100 and wght=449.

- Contour order differs in glyph 'uni023E.ss01.BRACKET.varAlt01': [0, 1, 2, 3] in wght=100, [1, 0, 2, 3] in wght=449.

[code: interpolation-issues]

⚠️ WARN Is there kerning info for non-ligated sequences?
  • ⚠️ WARN

    GPOS 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
  • ⚠️ WARN

    The 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?
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

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


    [code: lacks-article]
⚠️ 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+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

[code: unreachable-subsetting]
⚠️ WARN On a family update, the DESCRIPTION.en_us.html file should ideally also be updated.
  • ⚠️ WARN

    The DESCRIPTION.en_us.html file in this family has not changed in comparison to the latest font release on the google/fonts github repo.
    Please consider mentioning note-worthy improvements made to the family recently.


    [code: description-not-updated]
⚠️ WARN Ensure soft_dotted characters lose their dot when combined with marks that replace the dot.
  • ⚠️ WARN

    The 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
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    This font file does not have a 'meta' table.


    [code: lacks-meta-table]
ℹ️ INFO List all superfamily filepaths
  • ℹ️ INFO

    ofl/montserrat


    [code: family-path]
ℹ️ 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-Italic[wght].ttf
Dehinted Size 740.0kb
Hinted Size 744.5kb
Increase 4.5kb
Change 0.6 %
[code: size-impact]
ℹ️ 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 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 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 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 9.000']


    [code: version-not-detected]
[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-nonhinting script.


    [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)
[code: failed-language-shaping]
  • ⚠️ WARN

    GF_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)
[code: warning-language-shaping]
⚠️ WARN Check glyphs in mark glyph class are non-spacing.
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    Interpolation 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?
  • ⚠️ WARN

    GPOS 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
  • ⚠️ WARN

    The 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?
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

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


    [code: lacks-article]
⚠️ 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+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

[code: unreachable-subsetting]
⚠️ WARN Ensure soft_dotted characters lose their dot when combined with marks that replace the dot.
  • ⚠️ WARN

    The 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
  • ⚠️ WARN

    The 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.
  • ⚠️ WARN

    This font file does not have a 'meta' table.


    [code: lacks-meta-table]
ℹ️ INFO List all superfamily filepaths
  • ℹ️ INFO

    ofl/montserrat


    [code: family-path]
ℹ️ 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 %
[code: size-impact]
ℹ️ 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 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 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 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 9.000']


    [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-ital', 2)

[code: summary]

Summary

💥 ERROR ☠ FATAL 🔥 FAIL ⚠️ WARN ⏩ 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-- Upstream is working on it Designer is making changes in the upstream repo I Font Upgrade P2
Projects
Status: PR GF
Development

Successfully merging this pull request may close these issues.

Update Montserrat with SSA glyphset
3 participants