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

Decorations don't scale with editor zoom #78

Closed
brxck opened this issue Jul 7, 2021 · 13 comments
Closed

Decorations don't scale with editor zoom #78

brxck opened this issue Jul 7, 2021 · 13 comments
Labels
bug Something isn't working

Comments

@brxck
Copy link
Collaborator

brxck commented Jul 7, 2021

I'm on 5271ce3 and Ubuntu 20.10.

VS Code info:

Version: 1.57.1
Commit: 507ce72a4466fbb27b715c3722558bb15afa9f48
Date: 2021-06-17T13:26:56.255Z
Electron: 12.0.7
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: 8.9.255.25-electron.0
OS: Linux x64 5.8.0-59-generic

No Zoom

no zoom

5x Zoom

5x zoom

@pokey pokey added the bug Something isn't working label Jul 9, 2021
@brxck
Copy link
Collaborator Author

brxck commented Jul 16, 2021

This is the case on my new machine as well 😢 Ubuntu 21.04.

@pokey
Copy link
Member

pokey commented Jul 16, 2021

Weird! I believe that zoom level is a setting, so we could maybe read zoom level and adjust based on that.

Strange that it only happens for some VSCode setups. Does it happen on a clean VSCode instance with no extensions installed?

We could have a setting to toggle this correction and then mention it in known issues

@pokey
Copy link
Member

pokey commented Sep 11, 2021

@brxck when you get a minute, would be amazing if you could check whether #273 fixes this one

@brxck
Copy link
Collaborator Author

brxck commented Sep 11, 2021

@pokey unfortunately not 😦

Screenshot from 2021-09-11 10-08-24

Screenshot from 2021-09-11 10-08-45

@AndreasArvidsson
Copy link
Member

AndreasArvidsson commented Dec 10, 2021

@brxck Is this still a problem?

@AndreasArvidsson AndreasArvidsson self-assigned this Dec 11, 2021
@brxck
Copy link
Collaborator Author

brxck commented Dec 11, 2021

Yep, same behavior. Let me know if I can provide any other info/help!

image

@AndreasArvidsson
Copy link
Member

This is so strange. We have tested this on the windows, linux, and mac and it works on all three except for on your computer.

@AndreasArvidsson AndreasArvidsson removed their assignment Jan 5, 2022
@brxck
Copy link
Collaborator Author

brxck commented Feb 24, 2022

Well I just realized something about this! Come to find out that there are two types of zoom actions in vscode.

workbench.action.zoomIn scales all UI. Cursorless deals with this as expected.

editor.action.fontZoomIn scales only the editor font. This will trigger the issue. These actions don't have any keybindings by default, but they are triggered on ctrl + mousewheel over the editor.

@pokey
Copy link
Member

pokey commented Feb 24, 2022

ohh weird. and that's different from increasing the font size?

@brxck
Copy link
Collaborator Author

brxck commented Feb 24, 2022

Yep! Cursorless responds fine to font size changes. And apparently mousewheel zoom is a setting that is off by default? Don't remember turning that on... No wonder I'm the only person who's run into this 😅

@pokey
Copy link
Member

pokey commented Feb 24, 2022

Hmm. I wonder if we should fix this one or just add a note to https://github.com/cursorless-dev/cursorless-vscode/wiki/Troubleshooting

@brxck
Copy link
Collaborator Author

brxck commented Feb 24, 2022

Hardly seems worth fixing at this point. Especially after a brief poke around the extension API didn't yield an obvious way to get at this zoom level.

@pokey
Copy link
Member

pokey commented Feb 24, 2022

@pokey pokey closed this as completed Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants