search

TypeTogether / Playwrite

Sensei primary repository.
SIL Open Font License 1.1
93 stars 3 forks source link

⚠ Check GDEF mark glyph class doesn't have characters that are not marks. #11

Closed vv-monsalve closed 9 months ago

vv-monsalve commented 11 months ago

E.g.

⚠ WARN: Check GDEF mark glyph class doesn't have characters that are not marks. (com.google.fonts/check/gdef_non_mark_chars)
> >Glyphs in the GDEF mark glyph class become non-spacing and may be repositioned if they have mark anchors. > >Only combining mark glyphs should be in that class. Any non-mark glyph must not be in that class, in particular spacing glyphs. > * ⚠ **WARN** The following non-mark characters should not be in the GDEF mark glyph class: U+00B7 [code: non-mark-chars]
josescaglione commented 11 months ago

Kindly provide a full list. We will sort it out

vv-monsalve commented 11 months ago

This is the only one reported so far. If any other is reported, I'll add it here.

vv-monsalve commented 10 months ago

This is still reported for the following families at commit 1b4d055.

List of reports including the warning - qaPlaypen/FB-reports-1b4d055/qaPlaypen-AUS-SA.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-AUS-TAS.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-BEL-VLG.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-CAN.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-CHI.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-COL.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-DEU-Gru.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-DEU-LA.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-DEU-SAS.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-DEU-VA.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-DNK-looped.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-DNK-unlooped.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-FRA-Mod.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-FRA-Trad.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-HRV-Lefthand.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-HRV.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-IRL.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-ITA-Mod.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-ITA-Trad.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-MEX.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-NLD.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-NZL.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-PER.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-POL.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-POR.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-SVK.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-USA-Trad.md - qaPlaypen/FB-reports-1b4d055/qaPlaypen-ZAF.md
josescaglione commented 9 months ago

This is fixed