-
Notifications
You must be signed in to change notification settings - Fork 800
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
GoTo Line has a pause after hitting "return" (due to colorization?) #3011
Comments
I'm afraid we don't control this. |
Yeah I was wondering. Maybe it is just incentive for us to make the appropriate colorizer faster, or submit a roslyn bug to delay colorization |
I think something's not right on our side. For comparison open ProjectNode.cs in ProjectSystem.Base. Over 6k lines and go to line is instantaneous. |
The only thing I can think is syntactic colorization, which is synchronous in Roslyn (for a reason I don't fully understand, something like "we made (that's how I see it, maybe I'm wrong) |
I suspect it's the UI of that go to popup that blocks and stutters for some reason. Try making large jumps around the code by right clicking scrollbar and selecting "scroll here" and it's always snappy. |
My VFT is broken, see the latest issue. Cannot test. |
I think it must be that. If you go to the last line of a large file, it's slow. Then back to the top, and try again to the last line, and its fast. That strongly seems to indicate colorization. |
I reproduced it on tast.fs, it took about 2-3 seconds each time I went to line #5000. dotTrace shows this: After about a minute this stopped happening and it jumped instantly (no CPU load were for all that time). |
@vasily-kirichenko I can't see anything causal on the dotTrace info - can you? Thanks |
That's the problem :) |
cc @Pilchie |
I tried again and the pause seems much reduced now in large files. This is an old issue so closing |
Repro steps
Expected behavior
Goes to the line very fast, then colorizes
Actual behavior
Takes a while, I think it is colorizing the file. (Is it waiting for a semantic colorization? Or just slowly doing syntax colorization)
Known workarounds
Be patient and just wait
The text was updated successfully, but these errors were encountered: