-
Notifications
You must be signed in to change notification settings - Fork 30.7k
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
Scrolling is not smooth but lags #61824
Comments
@gurpreetshanky What is your windows version? Copy it from running winver.exe |
Version 1809 ( OS Build 18267.1001) |
@gurpreetshanky interesting, that should be Windows 10 RS5. Can you give more details about:
|
This is the Dell XPS 15 with Microsoft Precision touchpad. This is the same bug which I have linked #13612. Most probably, it has returned in insider version of Windows. |
@gurpreetshanky that would be bad, any chance you could test the non-insiders version? |
Yes this problem is not present in stable version on turning smoothScrollingWorkaround true. Has smoothScrollingWorkaround setting is removed in insiders version? I cannot find it. |
@gurpreetshanky yes I removed it assuming that this issue was fixed in Windows 10 RS5. It is unlikely I would bring it back, we should rather find out why this is broken again in Windows insiders. I wonder if your insiders version is somehow behind the RS5 update. I actually meant to ask if you could try a Windows Stable version. |
@bpasero No mine is fast ring(18267), so its after the RS5 update. This bug definitely came back in new Windows Insiders version. |
I will follow up with the developer that pushed the fix to ask if this possibly had to be reverted for some reason |
@gurpreetshanky, can you please collect a trace of the issue?
|
@rossnichols here is the file https://1drv.ms/u/s!AgmFl0OpBO1agYtxQUmHZHEtPL8dlg |
@gurpreetshanky thanks! Root cause is known, I'll work with Ben on getting VSCode fixed. |
@korpong753, you don't need to follow the instructions, I have all the information I needed. You'll just have to wait until VSCode has an update that fixes it. |
@rossnichols, I'm not sure, Is it bug of VSCode or not, Once we have checked and updated window 10 pro, now we can use touchpad with smoothly without lag problem anymore. This is window version that we updated. (https://support.microsoft.com/th-th/help/4462933/windows-10-update-kb4462933) |
@korpong753, can you collect a trace using the instructions above? |
@rossnichols I can confirm scrolling is much better with a precision touchpad after the KB4462933 update with minimal lagging (still lags a little bit, but it's not affected by VSCode's focus change, AFAICT). Windows & VSCode both on stable. Edit: This is the related change in the update:
|
@gurpreetshanky can I ask you if the scroll-lag workaround setting in VSCode fixes the issue for your? Just configure:
|
@bpasero Yes it has fixed the issue for me by these workarounds. |
@gurpreetshanky @korpong753 @HazemAM can you help me to test a new build with the fix from https://chromium-review.googlesource.com/c/chromium/src/+/1299342 to see if it works now without having Steps:
|
@bpasero Honestly I tried both the Insiders versions (normal and above provided one) and checked multiple times. The workaround was not required in both situations. |
@gurpreetshanky great, thanks for confirming. By now the current insiders version and the one I linked will both have the fix included 👍 |
Closing this issue given that we plan to release VSCode stable early February with Electron 3.x. If you want to benefit from the fix already, consider to use our insiders version that already contains the fix: https://code.visualstudio.com/insiders/ |
Steps to Reproduce:
Does this issue occur when all extensions are disabled?: Yes
This is a duplicate of #13612 but I couldn't post it there as it was closed.
Comment from @bpasero:
Depends on electron/electron#8960 (comment)
The text was updated successfully, but these errors were encountered: