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

Add MVAR test files exercising all global metrics #666

Merged
merged 1 commit into from
Dec 18, 2023
Merged

Conversation

anthrotype
Copy link
Member

This PR includes an 'MVAR Test' VF in Glyphs3, Glyphs2 and DesignSpace formats to be used for testing #648.
I separated this to avoid that other PR to get too noisy. This one is expected to be merged first, as the other depends on it for the tests.

The VF has one 'slnt' axis and 3 masters, one upright (default) and two backward- and forward-slanted masters; in the back-slanted master, all the variable global metrics are shifted by -5 units, in the forward-slanted master they are shifted +5 (the numbers don't make sense from a design POV it's just to simplify testing).

The purpose of the test files is to make sure that all the global metrics defined in the source files make it to the MVAR.

The reason the DS/UFO sources are not auto-generated from the .glyphs ones is because there are some variable global metrics that can be set in UFO fontinfo.plist but cannot be overridden in .glyphs sources: i.e. the hhea caret slope rise/run/offset; in Glyphs, these are always derived based on the italic angle and can't be controlled directly.

JMM

The 'MVAR Test' VF (in Glyphs3, Glyphs2 and DS formats) has one 'slnt' axis and 3 masters, one upright (default) and two back- and forward-slanted masters; in the backslanted master, all the variable global metrics are shifted -5, in the forward-slanted master they are shifted +5 (this to simplify testing).
The purpose of the test files is to make sure that all the global metrics defined in the source files make it to the MVAR.
@rsheeter rsheeter added this pull request to the merge queue Dec 18, 2023
Merged via the queue into main with commit 0174779 Dec 18, 2023
10 checks passed
@rsheeter rsheeter deleted the mvar-test-files branch December 18, 2023 18:15
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.

2 participants