-
Notifications
You must be signed in to change notification settings - Fork 95
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
Overlapping components don't get merged in static ttf #813
Comments
decomposed how, by whom? Perhas you are inserting a ufo2ft filter to decompose specific composite glyphs that you know contain overlapping components (like this)? If so, make sure to define the filter as a pre-filter, i.e. one that is performed before the default filters (which include overlap removal for statics). You do that by setting <key>pre</key>
<true/> |
I'm not familiar with the tool, or whether/how it calls fontmake under the hood, whether it builds static fonts from the sources directly (e.g. |
Ah ok thanks for the explanation, in that case I cc @m4rc1e cause it is probably be an issue from the builder then. |
I wondered about this this morning. I don't quite understand it. We remove overlaps from overlapping contours because certain environments have rendering issues with overlaps. But surely those environments would also have rendering issues with overlapping components too? So I'm not sure why it makes sense to remove overlaps from contours but not components. (I was actually thinking about this the other way around: if overlapping components are actually supported in rendering environments, then we can "remove overlaps" from contours by simply turning them into components, avoiding all the fiddly pathops math.) |
We just noticed while dealing with League Spartan
In static ttf, overlapping components are decomposed but outline don't get merged (and so they overlap). If static ttf have merged contours by default, so decomposed components should too, right?
Building process uses gftools builder with direct source as argument. Before and after export:
cc @emmamarichal (who is currently working with me to onboard fonts in Google Fonts)
The text was updated successfully, but these errors were encountered: