-
Notifications
You must be signed in to change notification settings - Fork 29.9k
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
Proportional fonts are rendered as fixed width fonts in the terminal #35681
Comments
Proportional fonts are just as valid as monospace fonts for coding. It would be nice to have better rendering in the terminal since the extra spacing makes proportional fonts difficult to read (they seem to have even wider spacing than monospace fonts). For now I'm setting |
Update settings to "terminal.integrated.fontFamily": "Monaco", it works for me. |
Resetting the font does not solve the problem. |
You are right. It is a workaround, you have to use a monospaced font in the terminal. |
I set the terminal font for "terminal. Integrated. FontFamily" is still doesn't work, so that the terminal can make me look very pain |
@Hideer you need to use a monospace font, variable width fonts are not supported within the terminal. |
Hey, I'm in need of this |
@matt3224 decimal letter spacing would only be added after proportional font support is added. Font rendering is currently done on a canvas and every character is drawn on a grid and their texture cached to boost performance, because of this their x,y coordinates need to be integers (otherwise they would be blurry). |
@Tyriar how far along is the proportional font support? |
@matt3224 it's not planned |
@Tyriar But you closed #35695 in favor of this one and now it's not planned? As far as I can see on xtermjs/xterm.js#1044, letter spacing is supported by xterm, right? So could we at least get that on VS Code? I mean, I just wanted to have 1px more between each character because right now they just feel weird. And that 1px between each character would render the terminal much more closely to how Windows Command Prompt renders. It's just 1px, it's an integer and their texture could still be cached without a performance hit, right? |
@rfgamaral it was actually an oversight that I didn't bring in I'll add this in the May release as that's trivial. Tracking in #49566 to make sure it gets done/verified. Sorry about the hassle and thanks for calling this out. |
@Tyriar No problem, thank you :) |
I had the same issue. because I imported settings from someone else. (I'm using linux, imported settings from a mac) |
@dulara1994 remember you can also use font fallbacks, so you can have |
Closing as font fix, you should be using a monospace font in a terminal. If you're experiencing this issue you can fix it by configuring the |
Steps to Reproduce:
Reproduces without extensions: Yes/No
The text was updated successfully, but these errors were encountered: