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

[Audit] FB Report Build 1.08: Ensure the font supports case swapping for all its glyphs. #343

Closed
kateliev opened this issue Aug 7, 2024 · 2 comments
Assignees
Labels
FontBakery Fail or Error from FontBakery tests help wanted Extra attention is needed invalid Problem description wrong? Issue is no longer relevant or already fixed? wontfix This will not be worked on

Comments

@kateliev
Copy link
Member

kateliev commented Aug 7, 2024

@tphinney the following is a bit ridiculous FAIL connected to #341. I would generally ignore it, but still i need your opinion...

Ensure that no glyph lacks its corresponding upper or lower counterpart (but only when unicode supports case-mapping).

🔥 ScienceGothic[YOPQ,slnt,wdth,wght].ttf
  • 🔥 FAIL

    The following glyphs lack their case-swapping counterparts:

    Glyph present in the font | Missing case-swapping counterpart -- | -- U+24B9: CIRCLED LATIN CAPITAL LETTER D | U+24D3: CIRCLED LATIN SMALL LETTER D

    [code: missing-case-counterparts]

@kateliev kateliev added the help wanted Extra attention is needed label Aug 7, 2024
@kateliev kateliev added the FontBakery Fail or Error from FontBakery tests label Aug 7, 2024
@tphinney
Copy link
Collaborator

tphinney commented Aug 7, 2024

LOL. Yeah we can ignore this.

Mentioned in discussion with GF folks as well. hah!

@tphinney tphinney closed this as completed Aug 7, 2024
@github-project-automation github-project-automation bot moved this from To do to Done in Science Gothic VF Aug 7, 2024
@tphinney tphinney added invalid Problem description wrong? Issue is no longer relevant or already fixed? wontfix This will not be worked on labels Aug 8, 2024
@tphinney
Copy link
Collaborator

tphinney commented Sep 3, 2024

Actually, turned out they had already fixed this. I was one release behind on Fontbakery....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FontBakery Fail or Error from FontBakery tests help wanted Extra attention is needed invalid Problem description wrong? Issue is no longer relevant or already fixed? wontfix This will not be worked on
Projects
Status: Done
Development

No branches or pull requests

2 participants