TypeTogether / Playwrite

Sensei primary repository.
SIL Open Font License 1.1
85 stars 3 forks source link

Roman and Italic VFs #29

Closed vv-monsalve closed 2 months ago

vv-monsalve commented 7 months ago

@josescaglione After reviewing the current status of italic fonts produced with an slnt axis, we've confirmed the requirement to split the Variable Fonts into Roman + Italics, as it is still stated in our Guide.

This is necessary to ensure the Italic companions are visible and accessible from the Catalog and to receive the static counterparts in the .zip file for each font.

vv-monsalve commented 5 months ago

The fonts in the lang-build branch at commit 658f99f.

@simoncozens this is still missing in builder2

simoncozens commented 5 months ago

Or is it?

simon@Simons-Air ~/Downloads/Playwrite-fonts-3 $ ls variable/PlaywriteGB*
'variable/PlaywriteGBJ-Italic[wght].ttf'
'variable/PlaywriteGBJ[wght].ttf'
'variable/PlaywriteGBS-Italic[wght].ttf'
'variable/PlaywriteGBS[wght].ttf'
vv-monsalve commented 5 months ago

They were not included in the local make build run.

After a private chat conversation with Simon, he said this is due to a dependency version. Simon is going to pin it later on when Builder2 has reached a more stable development process :)

vv-monsalve commented 3 months ago

The current builder2 branch is creating separate fonts but I'm keeping this issue open until we ensure the STAT tables are working ok.

vv-monsalve commented 2 months ago

This has been solved. Closing it here.