-
Notifications
You must be signed in to change notification settings - Fork 30.8k
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
Integrated terminal renders vertical gap between box characters #110039
Comments
Cannot reproduce with the font Cascadia Mono. Does the same issue show up for you with that font? |
Using the webgl renderer might help with this: But I think what you're really after is to not draw the box characters from the font at all and draw the box characters pixel perfect which is captured in xtermjs/xterm.js#2409 |
Interesting, I tried different fonts to rule out a problem with a particular font's metrics. Roboto Mono, DejaVu Mono, and Consolas all showed the same issue. However, Cascadia Mono does indeed render without the vertical gaps.
It did not.
It looks promising. But looking through the comments there, I remembered someting else that might be related. There seems to be some kind of weird clipping in VS Code's integrated terminal (in addition to a difference in font weight, even though I did not specify the weight in either program): |
@l-c-g Windows Terminal may indeed do pixel perfect rendering. The clipping should be restricted to the canvas renderer (unless you're using an abnormally large glyph). |
Oh, I always thought that Windows Terminal and terminal in VS Code used the same renderer. That's why I kept comparing the two. I don't know why I started thinking that, though, and I never checked it.
Yes, I switched to the DOM renderer and the clock is no longer clipped at the top. |
@l-c-g nope, WT is written in C++ from scratch and VS Code uses xterm.js. The only component they share is "conpty" that handles launching the shell. Since we're doing issue grooming at the moment I'm going to close this in favor of xtermjs/xterm.js#2409. The clock being cut off should be restricted to the canvas renderer which we want to remove #106202 |
Steps to Reproduce:
Does this issue occur when all extensions are disabled?: Yes
Additional information:
My Windows is set to 150% display scaling, the issue also occurs at 100% scaling. Interestingly, my Windows Terminal is configured to a font size 10, whereas VS Code needs 14 to achieve a similar font size.
The text was updated successfully, but these errors were encountered: