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

Bad vertical alignment of u #76

Closed
johannschopplich opened this issue Sep 24, 2018 · 11 comments
Closed

Bad vertical alignment of u #76

johannschopplich opened this issue Sep 24, 2018 · 11 comments
Labels
bug Something that is now the way it's supposed to be

Comments

@johannschopplich
Copy link

johannschopplich commented Sep 24, 2018

Describe the bug
The character u is missaligned vertically

To Reproduce
Install hinted TTF version of Inter UI on an Windows machine and write something down.

Screenshots
grafik

Environment

  • Version: Inter UI 3.0 RC1 (TTF hinted)
  • OS: Windows 10
@rsms
Copy link
Owner

rsms commented Sep 24, 2018

Yikes! Did you try the unhinted version? Also, are you using the traditional font files or the “variable” ones?

@johannschopplich
Copy link
Author

I will try the unhinted version for comparison. No, I'm using the traditional hinted TTF version.

@johannschopplich
Copy link
Author

Tested it. Seems like a hinting problem. I guess there's nothing you can do then.

@rsms rsms added the bug Something that is now the way it's supposed to be label Sep 24, 2018
@rsms
Copy link
Owner

rsms commented Sep 24, 2018

We can definitely fix this :–)

Some glyphs are composed from other glyphs via something called "components" — a sort of hyperlink. This is the case with "u", which is an instance of the "n" component mirrored on the Y axis. However, I'm suspecting that autohint introduces TTF hints on the "n" glyph which flow over to the "u" glyph, but as "u" is transformed to mirror, the TTF hints will be off.

Here's a build with "u" as a regular glyph rather than a component:
Inter-UI-3.0-7bf2380cf3.zip

@johannschopplich
Copy link
Author

johannschopplich commented Sep 24, 2018

Wow, I didn't expect the hinting process to be that complex. Everything I threw in ttfautohint just worked. Thanks for the component introduction. 🙂

Your build does look much better:
grafik

But now the "v" looks more dominant than the "r". Like these glyphs have different x-heights. Which they don't... Quite a task. Hinting options.

@rsms
Copy link
Owner

rsms commented Sep 25, 2018

Can you share how you reproduce the above? I’m unable to repro that rendering on Windows 10.

@johannschopplich
Copy link
Author

johannschopplich commented Sep 25, 2018

It seems that Thunderbird 60.0b11 (which I used to test) renderes hinted TTFs different. I retested in MS Office Word and Inter UI renders fairly fine even in larger sizes.

@rsms
Copy link
Owner

rsms commented Sep 25, 2018

Interesting. It seems it may be an issue with Thunderbird. There are at least two open issues related to ClearType text rendering. https://bugzilla.mozilla.org/show_bug.cgi?id=1374225

If it looks good on Windows (apps on Windows that use the OSes text rendering) I'm going to call it and close tihis ticket. Let me know if that is not the case and we can reopen this and investigate rendering on Windows' :–)

@rsms rsms closed this as completed Sep 25, 2018
@johannschopplich
Copy link
Author

johannschopplich commented Sep 25, 2018

Sounds like a plan. Thanks for the thorough research. Looking forward to RC2 now. 😄

@rsms
Copy link
Owner

rsms commented Sep 25, 2018

RC2 should be released tomorrow.

@rsms
Copy link
Owner

rsms commented Sep 27, 2018

RC 2 has been released and should fix this issue.
https://github.com/rsms/inter/releases/tag/v3.0-rc2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that is now the way it's supposed to be
Projects
None yet
Development

No branches or pull requests

2 participants