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

added com.adobe.fonts/check/find_empty_letters #2460

Merged
merged 2 commits into from Apr 10, 2019

Conversation

cjchapman
Copy link
Member

@cjchapman cjchapman commented Apr 10, 2019

Description

Font language, script, and character set tagging approaches typically have an underlying assumption that letters (i.e. characters with Unicode general category 'Ll', 'Lm', 'Lo', 'Lt', or 'Lu', which includes CJK ideographs and Hangul syllables) with entries in the 'cmap' table have glyphs with ink (with a few exceptions, notably the Hangul "filler" characters).

This check is intended to identify fonts in which such letters have been mapped to empty glyphs (typically done as a form of subsetting). Letters with empty glyphs should have their entries removed from the 'cmap' table, even if the empty glyphs are left in place (e.g. for CID consistency).

To Do

  • update CHANGELOG.md
  • wait for checks to pass (except github/pages, which is stuck)
  • request a review

Font language, script, and character set tagging approaches typically have an underlying assumption that letters (i.e. characters with Unicode general category 'Ll', 'Lm', 'Lo', 'Lt', or 'Lu', which includes CJK ideographs and Hangul syllables) with entries in the 'cmap' table have glyphs with ink (with a few exceptions, notably the Hangul "filler" characters).

This check is intended to identify fonts in which such letters have been mapped to empty glyphs (typically done as a form of subsetting). Letters with empty glyphs should have their entries removed from the 'cmap' table, even if the empty glyphs are left in place (e.g. for CID consistency).
Lib/fontbakery/profiles/adobefonts.py Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@cjchapman cjchapman force-pushed the cjc-add-check-find-empty-letters branch from 4d3d72f to 8d4403a Compare April 10, 2019 17:58
@felipesanches felipesanches merged commit 784d330 into master Apr 10, 2019
@cjchapman cjchapman deleted the cjc-add-check-find-empty-letters branch April 10, 2019 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants