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

[STAT] Added ranges to weight values #204

Merged
merged 9 commits into from May 4, 2020
Merged

[STAT] Added ranges to weight values #204

merged 9 commits into from May 4, 2020

Conversation

arialcrime
Copy link
Contributor

@arialcrime arialcrime commented Apr 29, 2020

The fonts now include range definitions for the weight and optical size values in the stat table. The weight implementation is in tune with what other fonts are doing. Having something similar for the optical size would also make a lot of sense. So as a “discussion starter” I added range values there too. A value like 9pt[36] Black S100 maybe also questions the current Google Fonts Spec when it comes to naming a optical size axis values. Feel free to investigate the current STAT table with Samsa.

The missing 0 value for the wonk axis is due to the current limitations within statmake where you can only add values that are also present in the fvar (named instances). Hoping @madig can find some time for an update soon. ;-)

The fonts are installable without any unwanted duplicates on macOS Catalina and Windows 10, and works with some minor restrictions in InDesign, Illustratur, and Sketch.

CC: @davelab6 @m4rc1e

@arialcrime arialcrime requested a review from sponcey April 29, 2020 14:34
@sponcey
Copy link
Collaborator

sponcey commented Apr 29, 2020

Interesting, that makes sense to define the opsz with a range in a similar way as the wght. I'll let @davelab6 comment before I merge.

also turn soften into format 3 to see if it works in Adobe software then (it does)
@sponcey sponcey merged commit 650329e into master May 4, 2020
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.

None yet

2 participants