-
Notifications
You must be signed in to change notification settings - Fork 29k
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
Blurry font after terminal resize #84194
Comments
Can you show a screenshot with the unblurry state?
This could literally not be further from the truth. |
It's not electron but rather an issue with canvas scaling in https://github.com/xtermjs/xterm.js that has been notoriously difficult to track down. If you zoom in you can see the x axis is probably 1px off which is causing the blurriness: Can you try repro using the xterm.js demo and create an issue there if you can still repro it, detailing your DPI (run window.devicePixelRatio in console) and Windows zoom factor (run make text size bigger in start menu)? |
I don't know what any of these fancy words mean. Seems like I would waste a lot of time and effort if I tried to figure out why this particular microsoft (also electron) app is buggy as always. And since the issue is closed, that means the problem has disappeared and your "Issues" are now manageable, right? Well, anyway I have: I tried running window.devicePixelRatio in the terminal, and it failed. Then I figured out you're probably talking about the javascript browser console (because this is almost a web app, right?). So my window.devicePixelRatio is 1.0416666269302368. |
Ok, this is the reason it's happening. Scaling the browser window outside of multiples of 100% can have 1-off errors and I don't think there's a way to resolve them on the web platform when you need pixel-perfect rendering in a canvas as from the web app's perspective you don't know whether it's going to round up or down. To my knowledge since I had looked into this in the past, you cannot fix this outside of using zoomLevel 0 due to limitations in the web platform. |
Yes, but when setting zoomLevel = 0, everything looks giant on my screen. Everyone else treats scaling problems as a bug, except UWP apps which were designed to be a mobile app language and look huge and dumb, and bad web apps, because they're trying to be a low-fidelity emulation of whatever happens to be popular. Other good text editors and IDEs (like sublime text) do not have this problem. This is the kind of issue that only web apps can afford to ignore, with the excuse that "you know, it's the web duh". |
Have you tried tweaking the editor font size? We also have the a highly upvoted feature request to change the workbench font size #519, eventually I'd like to see zoomLevel gone and replaced entirely with font size scaling instead of zooming the entire window so that we don't have to deal with this exact problem.
Yes it's a bug, but it's not actionable and that's how we manage our issues. What's the point of having issues open that cannot be fixed? |
It's entirely actionable, it's just that this is complicated, and as you said it will take effort and determination to fix, yet other apps seem to get it right somehow. |
I just said
|
If vscode UI was not huge and disproportionate on my computer I would not be changing the zoom level. |
|
Issue Type: Bug
With terminal open, do:
The terminal font is now blurry.
This is how it looks like:
When you undo this, the fonts gets unblurred.
Since you can't repeat the same options to have the terminal closed, to hide the sidebar again you have to right click the debug titlebar and select
Hide sidebar
.This since visual studio code's is an electron app and even its own devs don't use it, users have to endure terrible and annoying bugs like these.
VS Code version: Code 1.40.0 (86405ea, 2019-11-06T17:02:13.381Z)
OS version: Windows_NT x64 10.0.18362
System Info
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
metal: disabled_off
multiple_raster_threads: enabled_on
oop_rasterization: disabled_off
protected_video_decode: enabled
rasterization: enabled
skia_renderer: disabled_off
surface_control: disabled_off
surface_synchronization: enabled_on
video_decode: enabled
viz_display_compositor: enabled_on
viz_hit_test_surface_layer: disabled_off
webgl: enabled
webgl2: enabled
Extensions (14)
(1 theme extensions excluded)
The text was updated successfully, but these errors were encountered: