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.

Sign up for GitHub

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

Jump to bottom

Theta (θ, U+03B8) has wrong contour direction in single weight OTFs #117

Closed
NightFurySL2001 opened this issue Oct 27, 2021 · 5 comments
Closed

Theta (θ, U+03B8) has wrong contour direction in single weight OTFs #117

NightFurySL2001 opened this issue Oct 27, 2021 · 5 comments

Comments

@NightFurySL2001
Copy link

NightFurySL2001 commented Oct 27, 2021 •
edited
Loading

The inner countour of theta θ (U+03B8, CID+288) is inverted for all weights of all regions in Subset OTF.
image
@punchcutter
Copy link
Member

Not just all regions, but all fonts because they all come from a single source. I'm not sure how this happened, but will get it fixed.

@NightFurySL2001 NightFurySL2001 changed the title Theta (θ, U+03B8) has wrong contour direction in Subset OTF Theta (θ, U+03B8) has wrong contour direction in single weight OTFs Oct 27, 2021
@NightFurySL2001
Copy link
Author

NightFurySL2001 commented Oct 27, 2021 •
edited
Loading

Verified for single weight instaces OTF, but variable TTFs doesn't seem to have this issue.

@punchcutter
Copy link
Member

Variable fonts don't have overlaps removed. That's where it's happening.

@jimmymasaru
Copy link

I think the issue is because the exterior contour is not counterclockwise. It should be counterclockwise with the two internal clockwise contours. While most font engines can correct that, font editing software would render without correcting.

@punchcutter
Copy link
Member

punchcutter commented Oct 27, 2021 •
edited
Loading

The contours are perfectly fine in the source files. The tool used to remove overlaps has a bug.

@punchcutter punchcutter mentioned this issue Nov 21, 2021
Closed
@kaydeearts kaydeearts mentioned this issue Aug 3, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@punchcutter @jimmymasaru @NightFurySL2001