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

Overhaul language definitions in preparation for GF_Glyphset langdef-based overhaul #81

Merged
merged 23 commits into from
Jun 28, 2023

Conversation

yanone
Copy link
Contributor

@yanone yanone commented Jun 2, 2023

This PR removes various punctuation and other symbols from all languages in preparation of using the language definitions to check for language shaping in fontbakery.

Duplicate whitespace was also removed which applied to quite a few text sample as well.

Copy link
Contributor

@moyogo moyogo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is getting tricky. Some of these are not actually required and some of these are only required in some orthographies of a single language that has multiple orthographies.

Ligature ij is not required to support Dutch but may be nice to have for typesetting. Fonts don't have to have it to support Dutch. I would hard code this in glyphssets when handling Dutch, not in the language data base exemplar set. Maybe we need additional data to represent this in gflanguages.

Sharp s is used in some Low German orthographies but not in others. So fonts may support some orthographies of Low German even if they don't have it. For this we need to allow multiple orthographies in gflanguages.

I don't think these are blocking this PR but they'll need to be addressed at some point soon because they misrepresent language requirements.

@yanone
Copy link
Contributor Author

yanone commented Jun 5, 2023

@moyogo I removed the ij and IJ from Dutch.

I added the lowercase ß to Low German only because it already had an uppercase version defined previously, which on its own is certainly nonsensical.

@RosaWagner
Copy link
Contributor

I agree with this PR. I don't see any danger here in removing these glyphs.

My only note: the removal of hyphen2 and softhyphen resulted in a double space.

@RosaWagner
Copy link
Contributor

As @moyogo suggested also in #84 section sign could also be removed

@yanone
Copy link
Contributor Author

yanone commented Jun 9, 2023

Okay, I removed the section sign and all duplicate spaces

@yanone
Copy link
Contributor Author

yanone commented Jun 9, 2023

Today I will put together a document that outlines in which order the various repos need to be merged and releases cut, so please wait with merging.

@RosaWagner RosaWagner marked this pull request as ready for review June 21, 2023 16:57
Copy link
Contributor

@moyogo moyogo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Besides these comments, I think this can be merged.

@@ -2,3 +2,5 @@
*.egg-info
*__pycache__
Lib/gflanguages/_version.py

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove white space.

@@ -8,5 +8,5 @@ region: "LV"
exemplar_chars {
base: "A Ā B C Č D E Ē F G Ģ H I Ī J K Ķ L Ļ M N Ņ O Ō P R S Š T U Ū V Y Z Ž a ā b c č d e ē f g ģ h i ī j k ķ l ļ m n ņ o ō p r s š t u ū v y z ž"
auxiliary: "Q W X q w x"
marks: "◌̄ ◌̌ ◌̧"
marks: "◌̄ ◌̌ ◌̧ ◌ ̒"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding 0x0312 COMBINING TURNED COMMA ABOVE to Latvian (lv), Latgalian (ltg), Prussian (prg) is incorrect. While the glyph may be used as a component of ģ, the character is not used in those languages.

@moyogo moyogo merged commit 81ed497 into main Jun 28, 2023
4 checks passed
@moyogo moyogo deleted the overhaul-langdefs branch June 30, 2023 15:16
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.

3 participants