-
Notifications
You must be signed in to change notification settings - Fork 18
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
Missing codepoints in our Glyphsets WIP #235
Comments
First up, I implemented Auxiliary characters are not included in glyphsets except when instructed in the If they must be included, there are two good options: Either we raise their status in the language definitons from The first five above I would personally not include in glyphsets. To my knowledge, the contemporary conduct is to exclude encoded ligatures and have That being said, any font may choose to include any of these characters anyway despite them not being included in official glyphsets, so the glyphsets are never a barrier here. The And finally, the new |
But we could put |
We can also make |
I agree with most of the above points. I'll continue feeding the list as I review this font, and we can have a final revision afterward. The additional layer per script idea sounds good and is in line with what I mentioned yesterday about finding a way to define the glyphs necessary for all the scripts in one place so we do not repeat them on each glyphset. |
The list was longer than expected, so I filed an issue in the upstream repo to receive their and Denis' feedback, which could better inform our decisions about our Glyphsets. I left above a short list of the ones that we might need to add to some Glyphset and a couple of examples of auxiliary glyphs. |
Working on the subsetting of Playpen Sans pers script fonts, I've noticed some of the codepoints are not included in any of our Glyphsets.
After cross-referencing the Glyphsets with the gflanguages definitions, I've seen most cases correspond to auxiliary glyphs.
However, I'll collect the auxiliary plus other extra ones I'm identifying here so we can review them later on.
gcommaaccent
part of Latn CoreThe text was updated successfully, but these errors were encountered: