fonttools / fontbakery

🧁 A font quality assurance tool for everyone
https://fontbakery.readthedocs.io
Apache License 2.0
546 stars 99 forks source link

STAT check summary #3248

Open m4rc1e opened 3 years ago

m4rc1e commented 3 years ago

In pr https://github.com/google/fonts/pull/3304, @thlinard pointed out that the "Regular" axisValue must be elided.

For generating STAT tables, I make every axisValue which is a default in our Axis Registry elided, apart from the opsz axis. This seems to work nicely since it will correctly elide "Normal" (width), "Roman" (italic), "Default" (slant), "Proportional" (monospace) etc.

RosaWagner commented 3 years ago

This would be really helpful to have that check

This is linked to to-do list for STAT table check: https://github.com/googlefonts/fontbakery/issues/3090#issuecomment-729753965

These check allow a proper support of STAT table from different app, ie. to display instances' names correctly in drop down menu, and to allow proper linking of RIBBI styles.

Right now, checking for these 3 things force us to dump the STAT table, or check in Samsa.

davelab6 commented 3 years ago

Seems urgent! :)

thlinard commented 3 years ago

apart from the opsz axis

Elided default opsz value is much more elegant in applications with only named instances support (MS Office for Mac, Apple apps…).

Newsreader and Texturina have <Flags value = "2" /> on their default opsz value. See https://github.com/google/fonts/issues/3267