-
Notifications
You must be signed in to change notification settings - Fork 5
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
Kerning #14
Comments
@josescaglione, the following are the main findings after reviewing the basic kerning pairs for the 20 Countries of the Batch 1 to be published. Please review the following:
You can see all the Batch-1 fonts images here: Playwrite IndonesiaPlaywrite MexicoPlaywrite Vietnam |
Closing this issue with the above note. |
When the latest fonts on the |
I'll collect the reports we receive here so they can be revised on further font versions. |
added positive kerning between at and .cnt glyphs Checked all models. This changes affects: |
Confirmed it is working on fonts in |
Confirmed it is working on fonts in |
Confirmed it is working on fonts in |
@josescaglione I've finished the revision on the The kerning issue reported above would remain the only pending issue before merging the fixes branch into the main and continuing with the font upgrade. |
@josescaglione The above kerning case is solved in the fonts at commit Now, giving kerning a general overview, the following cases still need attention. Touching or nearly touching between
|
@josescaglione We have decided to keep the current fonts as they are to proceed with producing the prioritized guide variants. We will leave this issue open so that the bove reported cases can be reconsidered for a different font version in the future. Please merge the |
Hello @josescaglione! Hope you're doing well! I'm reviewing the first PR of the Playwrite upgrade, all looks good, thank you :) google/fonts#8826 I just saw something I wasn't sure about, but it's probably intentional! It's about the spacing of the “x”, which seems to be slightly different between masters. Could you confirm that this is ok for you, then I'll merge the PR :) Screen.Recording.2025-01-16.at.10.48.52.mov |
Please give kerning an overall revision and ensure the most common cases are covered. E.g.
Yo
Po
LY
LT
among othersThe text was updated successfully, but these errors were encountered: