-
Notifications
You must be signed in to change notification settings - Fork 4
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
Vertical Metrics #11
Comments
change in vertical metric comes from a couple of actual design changes:
note in frame one, ring is lighter and crashes into the character. I changed that to get what you see in frame two. same for the acute, it is steeper and heavier. both changes made to give diacritics a more consistent color and behavior across families—I didn't discover that the crashing ring was a terrible idea until dealing with Inline. |
This kind of height increase in an update is more typical when Vietnamese support is added. I believe the latest v metrics recommendation from @m4rc1e means that this doesn't cause an issue on the web, only on Android (which uses the bbox rather than v metrics, fml) |
Well, AFAIK @m4rc1e was really against the VM values changing when updating fonts, to avoid breaking anything down on documents or websites that are already using it. |
@vv-monsalve do the new vertical metrics alter the visual appearance? I don't mind if the winAscent and winDescent have been changed if fsSelection bit 7 has been enabled. |
@m4rc1e yep, you are right. Bit 7 is enabled and no visual appearance was altered. I couldn't test it yesterday, but doing now on cyreal.org it looks just fine. |
@vv-monsalve Did you also check Win Chrome? |
These are the Browser diffs and previews Result Big Shoulder Text Black |
Although, this sample doesn't display critical cases such as Vietnamese characters. A further check might be needed in that context. |
diffs look great. I broadened some kerning pairs based on what I saw going wrong in the city's COVID-19 video captions, this seems to reflect that OK. |
Vertical Metrics regarding Vietnamese can be tested with a basic HTML with Vietnamese texts, to ensure there would be no clipping, and to check it on Windows browsers. See this as a reference. |
When comparing the current static fonts with the one on the API, the vertical metrics are not the same. This sort of change must be avoided when updating a font to ensure fonts do not break existing documents or websites. (https://github.com/googlefonts/gf-docs/tree/master/Spec#scalable-font-production).
Data comparison between the .ttf on the API (string a), and the current static .ttf on the repository (string b):
Changes to Diacritics on the font are demanding higher values, but, this could be avoided by adjusting the size, weight, slope, or position of them in complex compound glyphs like the Aringacute or the Vietnamese.
The text was updated successfully, but these errors were encountered: