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

Characters are condensed #1046

Closed
mofux opened this issue Oct 10, 2017 · 6 comments
Closed

Characters are condensed #1046

mofux opened this issue Oct 10, 2017 · 6 comments
Labels
needs more info type/bug Something is misbehaving
Milestone

Comments

@mofux
Copy link
Contributor

mofux commented Oct 10, 2017

Since #1038 I am noticing that characters aren't as sharp as before. They also look not wide enough. Here is a direct comparision:

(expected) before #1038
screen shot 2017-10-10 at 20 47 43

(current) since #1038
screen shot 2017-10-10 at 20 49 17

I think only cached characters have this problem.

@mofux mofux added the type/bug Something is misbehaving label Oct 10, 2017
@mofux mofux added this to the 3.0.0 milestone Oct 10, 2017
@Tyriar
Copy link
Member

Tyriar commented Oct 10, 2017

Something doesn't look right here, this is at 100% zoom level in the image above for before:

screen shot 2017-10-10 at 12 42 59 pm

Something like this is I expect to see:

screen shot 2017-10-10 at 12 45 43 pm

Did you zoom into the screenshots or something?

Also the only remaining issue AFAIK is that the character don't use standard spacing which is only tracked in microsoft/vscode#35695 atm

@mofux
Copy link
Contributor Author

mofux commented Oct 10, 2017

Hmmm.... can't reproduce this blurriness you're seeing when comparing both pictures:

screen shot 2017-10-10 at 21 59 39

But look at the space between the af in the staff. Before there was 1px space, now there is 2px space between the right line of the a and the left arm of the f. The same is also visible when checking the gap between the s and the t.

screen shot 2017-10-10 at 22 20 22

@Tyriar
Copy link
Member

Tyriar commented Oct 10, 2017

So I can tell "right.png" is an older version because the top-right of the f is cut off. What commit and font is that? I see a different font as the default on macOS.

@mofux
Copy link
Contributor Author

mofux commented Oct 10, 2017

"right" is on commit 22af08d. You can replicate the settings on Mac by pasting this into your dev console of the demo:

term.setOption('fontFamily', 'monaco');
term.setOption('fontSize', 13);

If you look on the t or f strokes, you can clearly see that they are 1px smaller on the wrong.png

@Tyriar
Copy link
Member

Tyriar commented Oct 11, 2017

This might be the same as #1050

@mofux
Copy link
Contributor Author

mofux commented Oct 11, 2017

It is, your patch from Tyriar@09c9cb4 solves this issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs more info type/bug Something is misbehaving
Projects
None yet
Development

No branches or pull requests

2 participants