canonical / Ubuntu-Sans-fonts

Other
54 stars 4 forks source link

Name ID 25 missing from Ubuntu[wght,wdth] VF #36

Closed djrrb closed 1 year ago

djrrb commented 2 years ago

According to GF guide, Builder should automatically include Name ID 25 for InDesign:

The post-script name ID 25 for Adobe Indesign, which (for now) requires this scheme: FontName for the Roman font, and FontNameItalic for the Italic font. The Builder will set that up for you using default settings and/or your STAT table values.

This seems to be true in UbuntuMono, but not in Ubuntu, causing style linking to break in InDesign.

Of course I can add it after the fact but I’d rather keep the build process as clean as possible. I went in and added as much as I could to the stat description in the config. @guidoferreyra do you know what I’m missing?

guidoferreyra commented 2 years ago

STAT table in config file looks ok, the info related to Italic axis is added automatically by the builder and the values on the compiled font looks ok to me.

AFAIK postscript names for vf instances was not a requirement. Anyway Ubuntu has ID 25 (and others) with "Roman" and "Italic" on the name string, added by the builder.

In my experience style linking on InDesign is pretty buggy, for instnace changing to bold and changing back to regular only works if the font has Regular as its variable font origin, which is the case in this font.

I was expecting that linking in italics would work, but it doesn't work either. In the past I found this only works when:

So it might be something else here. I will keep triyng to find where the problem is.

Note: Style linking works properly on Figma

djrrb commented 2 years ago

The latest version I am building (weight-restructure branch) does not have name ID 25:

UbuntuBeta0.861[wdth,wght].ttf.zip

Screen Shot 2022-08-24 at 8 45 50 AM

Any idea how we can get this back?

guidoferreyra commented 1 year ago

I will be doing some tests on this today/tomorrow.

guidoferreyra commented 1 year ago

Tested weight-restructure branch and after making a small change to the designspace the font is built with ID 25 included.

Here the change, didn’t commited. imagen

Here the resulting name record. imagen

guidoferreyra commented 1 year ago

If you are ok with it, I can push the change to the branch

djrrb commented 1 year ago

You rock! Thanks so much for repairing that. Feel free to push.