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

Disabled features still interfere with automatic code #1026

Open
khaledhosny opened this issue Aug 29, 2024 · 2 comments
Open

Disabled features still interfere with automatic code #1026

khaledhosny opened this issue Aug 29, 2024 · 2 comments

Comments

@khaledhosny
Copy link
Collaborator

If a font has a disabled feature, we will still write a feature block to the UFO but with # disabled comment and all of its contents commented out. For feature with automatic code generation like kern or mark this will prevent ufo2ft from generating code for this feature. We could fix this by commenting out the whole feature block, but then going back from UFO to Glyphs will drop the feature (do we still care about this?).

@khaledhosny khaledhosny changed the title Disabled features still interfer with automatic code Disabled features still interfere with automatic code Aug 29, 2024
@anthrotype
Copy link
Member

going back from UFO to Glyphs will drop the feature

does ufo2glyphs un-comment the contents of these # disabled features?

We could fix this by commenting out the whole feature block

I guess we could do that only when minimal=True (fontmake sets that for compilation)

@khaledhosny
Copy link
Collaborator Author

going back from UFO to Glyphs will drop the feature

does ufo2glyphs un-comment the contents of these # disabled features?

Yes.

We could fix this by commenting out the whole feature block

I guess we could do that only when minimal=True (fontmake sets that for compilation)

Good idea.

@khaledhosny khaledhosny reopened this Aug 29, 2024
schriftgestalt pushed a commit that referenced this issue Oct 23, 2024
Partially fixes #1026 by
handling the case where we are building with fontmake.

When round tripping is desired, the generated UFO when built with ufo2ft
will still have the above issue.
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

No branches or pull requests

2 participants