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

Fix vertical spacing and height on Windows, Mac, Linux and in Java apps #48

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

glebd
Copy link

@glebd glebd commented May 10, 2015

Tweaked various heights, ascenders, descenders and line gaps to achieve a perfect vertical position of glyphs on Windows, OS X, Linux, including several notable apps like Java applications on Windows/Mac, Atom, Sublime Text, TextMate, Visual Studio, Xcode, Notepad++. In IntelliJ IDEA on Windows set line height to 0.9 for perfect results.

@Jackbennett
Copy link

I'm not anything to do with this project but I bet attaching some screenshots addressing a couple of problems this fixes might help the project owner out assessing the merge.

@glebd
Copy link
Author

glebd commented May 11, 2015

Good point, I'll do that.

@belluzj
Copy link
Owner

belluzj commented May 24, 2015

Indeed, I would to like to see some screenshots. Especially under Windows/Java, with dashes and underscores, because these characters tend to disappear depending on their vertical position... Thanks in advance

@fiendish
Copy link

fiendish commented Mar 7, 2017

I don't know if this was the same thing, but on my (Linux) system 11pt vertical placement and size is quite bad. See attached image. First line is 11pt. Second line is 12pt for comparison.

11vs12

[edit] Running the font through FontSquirrel's ttf generator (to get the non-loopy style set) with Auto-Adjust Vertical Metrics checked seems to fix it.

@maxhille
Copy link

Is this issue still valid?

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

Successfully merging this pull request may close these issues.

5 participants