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

Mutable binding syntax highlighting flashes too often when typing #10690

Closed
BlackHoleFox opened this issue Nov 4, 2021 · 6 comments
Closed
Labels
A-vscode vscode plugin issues Broken Window Bugs / technical debt to be addressed immediately S-unactionable Issue requires feedback, design decisions or is blocked on other work

Comments

@BlackHoleFox
Copy link

I'm not sure when this started since I only started to notice it and get annoyed by it earlier yesterday but as of recently the syntax highlighting that underlines mutable bindings in VSCode has been persistently annoying. Everytime I start and stop typing, every mutable binding on my screen briefly loses its underlining. This occurs even if I'm not changing any actual code

Even weirder, it only really happens when I type two characters consecutively. Typing one, pausing, then typing another doesn't trigger it. See below:

bfa7682f-e613-4871-bdaf-00d0edd77481

Let me know if there's anything else I can provide, but here are my relevant versions:

rust-analyzer version: a824768 2021-11-04 nightly
vscode-insiders version: 1.62.0-insider b3318bc0524af3d74034b8bb8a64df0ccf35549a
OS: Win 10

@Veykril
Copy link
Member

Veykril commented Nov 4, 2021

Can't reproduce this myself(vscode 1.16.1), this seems like either a VSCode bug to me, or maybe another extension of yours is interfering with things.

@jhgg
Copy link
Contributor

jhgg commented Nov 5, 2021

I can reproduce this in a larger workspace. It appears to be related to when salsa queries get cancelled and it sends the content modified error back to vscode. Not sure if the bug is in vscode's end or not however.

It affects all semantic tokens, not only mutable (which is the obvious one for most people who don't have semantic token customizations.)

Maybe regression on microsoft/vscode-languageserver-node#576 ?

@thomcc
Copy link
Member

thomcc commented Nov 5, 2021

I have this issue, except it happens for the color of documentation comments (flickering between the doc-comment and plain-comment color when I type). I'm assuming it's the same, since they're both related to semantic highlighting.

I have some issues with photo-sensitivity triggered by flashing, and when editing a lot of documentation the rapid flickering this caused felt like it was starting to trigger it. I'll find a way to work around it, but a fix would be greatly appreciated, since it could be a bigger issue for someone more sensitive to this than I am.

EDIT: actually, it seems like sometimes it's pretty much all the semantic highlighting in the file that flickers. Other times it's not as bad. Either way, it's very unfortunate :(. At least turning off semantic highlighting seems to have worked.

@Veykril
Copy link
Member

Veykril commented Nov 5, 2021

Okay I just tested it with the insiders build of VSCode(1.63.0-insider), and I can reproduce it there. So this seems to be a regression on their part as I can't reproduce it on 1.16.1.

@Veykril
Copy link
Member

Veykril commented Nov 5, 2021

Opened an issue on the vscode repo microsoft/vscode#136540

@Veykril Veykril added Broken Window Bugs / technical debt to be addressed immediately S-unactionable Issue requires feedback, design decisions or is blocked on other work labels Nov 5, 2021
@Veykril
Copy link
Member

Veykril commented Nov 11, 2021

The issue has been fixed upstream, so I assume the next insiders build should ship with it.

@Veykril Veykril closed this as completed Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-vscode vscode plugin issues Broken Window Bugs / technical debt to be addressed immediately S-unactionable Issue requires feedback, design decisions or is blocked on other work
Projects
None yet
Development

No branches or pull requests

4 participants