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

[glyphs] Handle additional custom params #1144

Open
wants to merge 1 commit into
base: glyphs-parse-params-to-plist
Choose a base branch
from

Conversation

cmyr
Copy link
Member

@cmyr cmyr commented Nov 26, 2024

This is based on #1143

This handles additional custom parameters, specifically openTypeHeadLowestRecPPEM, and the various hhea & vhea caret properties (vhea isn't handled yet, but we can at least parse it now in preparation).

These params are kind of funny because you don't seem to be able to add them via the glyphs.app UI? But they exist in some fonts, and were causing crater failures.

Specifically openTypeHeadLowestRecPPEM, and the various hhea & vhea
caret properties.

(vhea isn't handled yet, but we can at least parse it now in preparation)
@khaledhosny
Copy link
Contributor

These params are kind of funny because you don't seem to be able to add them via the glyphs.app UI?

You can add anything as Glyphs custom parameter. If Glyphs recognizes the name, it will give it the appropriate UI (e.g. a check box for boolean parameters), but otherwise you get a default text entry box that can have any value. Some of the parameters that Glyphs recognizes are hidden; if you search for them they will not show up, but if you go ahead and add it anyway it will get the appropriate UI.

They openType parameters, are, I think, present to parallel the corresponding UFO fontinfo keys, and I think Glyphs will use them when importing UFO files, rather than the “native” equivalent custom parameters.

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