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

after the update, the underscore is not displayed in the vscode terminal #211933

Open
AmIaLion opened this issue May 3, 2024 · 10 comments
Open
Assignees
Labels
regression Something that used to work is now broken terminal-rendering
Milestone

Comments

@AmIaLion
Copy link

AmIaLion commented May 3, 2024

Does this issue occur when all extensions are disabled?: Yes

  • VS Code Version:
    Version: 1.89.0
    Commit: b58957e
    Date: 2024-05-01T02:08:25.066Z
    Electron: 28.2.8
    ElectronBuildId: 27744544
    Chromium: 120.0.6099.291
    Node.js: 18.18.2
    V8: 12.0.267.19-electron.0
    OS: Linux x64 6.5.0-28-generic snap
  • OS Version: Linux nb-X 6.5.0-28-generic Send a smile > "Submit a bug" should open Github repo #29~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Thu Apr 4 14:39:20 UTC 2 x86_64 x86_64 x86_64 GNU/Linux

Steps to Reproduce:

  1. open vscode terminal
  2. send mkdir test_dir
    изображение

_ isn't displayed in some cases like pwd or git checkout -b test_branch
изображение
it not displayed while typing, but if ctrl+c, ctrl+v it shows.

@meganrogge
Copy link
Contributor

does changing terminal.integrated.gpuAcceleration help?

@meganrogge meganrogge added info-needed Issue requires more information from poster terminal-rendering labels May 3, 2024
@AmIaLion
Copy link
Author

AmIaLion commented May 4, 2024

does changing terminal.integrated.gpuAcceleration help?

It helps when set "canvas", previous setting was "auto"

@meganrogge meganrogge added regression Something that used to work is now broken and removed info-needed Issue requires more information from poster labels May 6, 2024
@kevin-presalytics
Copy link

kevin-presalytics commented May 7, 2024

I had this same problem when updating to the new version. Setting terminal.integrated.gpuAcceleration to "canvas" helped for me also.

@pindjur
Copy link

pindjur commented May 12, 2024

Same issue, Linux.
setting terminal.integrated.gpuAcceleration to anything other than "off" fixed it. Was "auto", but now when I updated this setting once, I could not reproduce the issue with "auto", just with "off"

@k98kurz
Copy link

k98kurz commented May 16, 2024

Looks like another regression. Same issue #123077 and #35901 before that. Changing font size fixes the display, though then you have to use a font size you did not want.

@bodik
Copy link

bodik commented May 16, 2024

same here, debian linux 12, "off" uses font variant without underscore visible, any other setting works fine (but also uses little bit different font, change is visible during changing the setting)

@jameshu15869
Copy link

jameshu15869 commented May 16, 2024

I'm seeing the same issue on Code 1.89.1 (dc96b83, 2024-05-07T05:16:23.416Z) on Linux Mint 21.3 - setting terminal.integrated.gpuAcceleration to canvas solved the issue for me

@Mendes11
Copy link

Just updated and started having this same issue. If set to canvas it works, anything else don't

Ubuntu Noble

@LincolnCB
Copy link

What has worked for me (each works separately):

  • Changing the font size to 13 or 15 (underscore is invisible on 14)
  • Changing the Terminal Integrated GPU Acceleration to on or canvas
  • Changing the Terminal Integrated Line Height to anything above 1.0625 (that seems to be the cutoff)
  • Changing the Terminal Font Family (every other one I tried seemed to work)
  • Changing the Terminal Integrated Font Weight to anything above 550 (close to bold, seems to be the cutoff)

I'm sticking with the Terminal Integrated GPU Acceleration to on right now, as it changes the least about my terminal's view.

@Tyriar
Copy link
Member

Tyriar commented May 30, 2024

This is happening is previously your setup would use the canvas renderer, which was removed as an option for auto in v1.89 and all together in v1.90. The canvas renderer includes some smarts about shifting underscores upwards when they sit particularly low which seems to be the case for some default fonts on Linux. This isn't actually a regression (xtermjs/xterm.js#4067), it's just being hit by more people since the canvas renderer is gone.

Since these issues seem restricted to Linux, I'm thinking for the upcoming release I'll force lineHeight to be 1.1 by default on Linux. Then in the next release come up with a more robust solution for these fonts in xtermjs/xterm.js#4067


Recommended workarounds:

  1. Set "terminal.integrated.gpuAcceleration": "on" (this might not work)
  2. Set "terminal.integrated.lineHeight": 1.1
  3. Change "terminal.integrated.fontFamily"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression Something that used to work is now broken terminal-rendering
Projects
None yet
Development

No branches or pull requests

10 participants