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

Deformed glyphs #255

Closed
greegus opened this issue Jan 19, 2017 · 8 comments
Closed

Deformed glyphs #255

greegus opened this issue Jan 19, 2017 · 8 comments
Labels

Comments

@greegus
Copy link

greegus commented Jan 19, 2017

Some glyphs have been parsed with misplaced point at the start/end of the bezier curve, and so the glyph appears to be deformed. This mostly occurs when the closing point is on the curve (not in the corner) typically on O, or 0 glyphs, but also some others (depending on the font).

Here is an example of such misplacement within O glyph in Futura Condensed ExtraBold font. The character on the left is outlined in Adobe Illustrator. On the right there is this character processed by opentype.js using Glyph Inspector (http://opentype.js.org/glyph-inspector.html) with same font loaded.

image

@cdave1
Copy link

cdave1 commented Feb 7, 2017

Here's a fairly rough patch I wrote to fix this, since I found similar problems with a lot of different fonts and it was driving me crazy.

text3386

@fpirsch
Copy link
Collaborator

fpirsch commented Feb 7, 2017

Thanks @greegus and @cdave1,
which version of Futura Condensed ExtraBold are you using ? I can't reproduce the problem with version 001.003 in OTF.

@greegus
Copy link
Author

greegus commented Feb 7, 2017

@fpirsch I'm using 006.000 TrueType

@fpirsch
Copy link
Collaborator

fpirsch commented Feb 7, 2017

Ah OK I see it now. Thanks @greegus !

@Jolg42
Copy link
Member

Jolg42 commented Mar 28, 2017

I can't find the exact Futura version cited before, any link or maybe another font with the same problem?

@Jolg42 Jolg42 added the bug label Mar 28, 2017
@greegus
Copy link
Author

greegus commented Mar 29, 2017

@Jolg42 futura-006.000.zip

@Jolg42
Copy link
Member

Jolg42 commented Mar 29, 2017

@greegus Thank you! Indeed I have the same problem, I'm gonna check that :)

@Jolg42
Copy link
Member

Jolg42 commented Mar 29, 2017

@greegus The fix is live on http://opentype.js.org/glyph-inspector.html Thanks to @cdave1 for the fix 😉

@Jolg42 Jolg42 closed this as completed Mar 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants