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

Vertical joining of box drawing characters has gaps #3029

Closed
tryptamic opened this issue Oct 2, 2019 · 6 comments
Closed

Vertical joining of box drawing characters has gaps #3029

tryptamic opened this issue Oct 2, 2019 · 6 comments
Labels
Area-Fonts Related to the font Area-Rendering Text rendering, emoji, complex glyph & font-fallback issues Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.

Comments

@tryptamic
Copy link

tryptamic commented Oct 2, 2019

Environment

Windows build number: Microsoft Windows [Version 10.0.18362.356]
Windows Terminal version (if applicable): 0.5.2681.0

Steps to reproduce

Run this litle command in the terminal using a cmd.exe tab to produce a little box:

echo ┌┐ & echo └┘

Expected behavior

The box should have perfectly joined lines on all sides. Living in a monospaced environment where the box drawing characters are used to draw trees, boxes and other stuf perfect line joining is expected

Actual behavior

The lines of the box are not vertically joined

image

@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Oct 2, 2019
@zadjii-msft
Copy link
Member

@tryptamic What font are you using?

What does this look like in conhost (launch cmd.exe directly) with the same font selected?

@zadjii-msft zadjii-msft added Area-Fonts Related to the font Area-Rendering Text rendering, emoji, complex glyph & font-fallback issues Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Oct 2, 2019
@egmontkob
Copy link

egmontkob commented Oct 2, 2019

Some terminals, including VTE and Konsole draw the U+2500..257F (or even up to 259F) glyphs manually, rather than taking them from the font. This prevents such problems (and also avoids ugly looking antialiasing). I haven't seen any monospace font that would just look perfect on its own.

Interestingly, independently from each other, both VTE and Konsole came up with the idea of defining the look of most of these codepoints as a 5x5 matrix (where how the given 5x5 grid is stretched to the character cell is not necessarily the same for thick lines vs. double lines).

VTE link1, link2

Update: Konsole recently changed not to use a 5x5 matrix but other means of drawing.

@DHowett-MSFT
Copy link
Contributor

I like the idea of having Terminal draw these itself; we've chatted about that internally and thought it was pretty clever.

This is also related to #455.

@tryptamic
Copy link
Author

tryptamic commented Oct 2, 2019

@tryptamic What font are you using?

What does this look like in conhost (launch cmd.exe directly) with the same font selected?

Consolas and here how it looks in in conhost:

image

In terminal with Cascadia Code it gets even worse because horizontal joining is then broken as well and vertical gaps got even larger as you can see in the screenshot

image

Why does it work in the old console without drawing it in a special way. Isn't it a simple issue of the wrong line height and in case of cascadia wrong glyphs?

@ghost ghost added Needs-Attention The core contributors need to come back around and look at this ASAP. and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Oct 2, 2019
@zadjii-msft
Copy link
Member

@tryptamic The legacy console and the Terminal use different renderers to display text. The Terminal's renderer is far more performant, but there are still many edge cases we need to work out, and this is one of them.

I think in general this falls under the umbrella of /dup #455 so I'm gonna close as a dupe.

@ghost
Copy link

ghost commented Oct 2, 2019

Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!

@ghost ghost closed this as completed Oct 2, 2019
@ghost ghost added Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements Needs-Attention The core contributors need to come back around and look at this ASAP. labels Oct 2, 2019
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Fonts Related to the font Area-Rendering Text rendering, emoji, complex glyph & font-fallback issues Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.
Projects
None yet
Development

No branches or pull requests

4 participants