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

Middle mouse button scrolling breaks when click+holding on a image within release notes. #61238

Closed
Steve-Fry opened this issue Oct 18, 2018 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug electron Issues and items related to Electron help wanted Issues identified as good community contribution opportunities upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded webview Webview issues
Milestone

Comments

@Steve-Fry
Copy link

Steve-Fry commented Oct 18, 2018

Issue Type: Bug

  1. Open VSCode
  2. Open release notes (Control+Shift+P --> Release Notes)
  3. Click and hold middle mouse button while mouse is over an image.

Expected behavior:

  • Screen will scroll according to offset of mouse postion from inital mousedown position.

Actual behavior:

  • No scrolling occurs.
  • All mousewheel based scrolling is not possible from this time on. This is only returned to normal functionality after switching tabs/reloading editor.

GIF Example
repro1
Events:

  • Left Click - Clear existing selected text
  • Middle Mouse Click on text
  • Middle Mouse Click on image
  • Middle mouse Click+Hold on text
  • Middle mouse Click+Hold on image

VS Code version: Code - Insiders 1.29.0-insider (9e83209, 2018-10-18T05:17:36.900Z)
OS version: Windows_NT x64 10.0.17134

System Info
Item Value
CPUs Intel(R) Core(TM) i5-8600K CPU @ 3.60GHz (6 x 3600)
GPU Status 2d_canvas: enabled
checker_imaging: disabled_off
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: disabled_software
rasterization: enabled
video_decode: enabled
video_encode: enabled
webgl: enabled
webgl2: enabled
Memory (System) 15.94GB (8.90GB free)
Process Argv
Screen Reader no
VM 0%
Extensions (1)
Extension Author (truncated) Version
python ms- 2018.9.0
Edit: Added explanation of steps on gif
@vscodebot vscodebot bot added the insiders label Oct 18, 2018
@vscodebot
Copy link

vscodebot bot commented Oct 18, 2018

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

@Steve-Fry
Copy link
Author

IMO not related to
#6302
#16514
#40985

@Tyriar Tyriar added the markdown Markdown support issues label Oct 23, 2018
@mjbvz mjbvz added webview Webview issues and removed markdown Markdown support issues labels Oct 23, 2018
@mjbvz mjbvz added the electron Issues and items related to Electron label Oct 23, 2018
@bpasero bpasero removed their assignment Oct 23, 2018
@bpasero bpasero added the upstream Issue identified as 'upstream' component related (exists outside of VS Code) label Oct 23, 2018
@vscodebot vscodebot bot removed the insiders label Oct 23, 2018
@T3sT3ro
Copy link

T3sT3ro commented Nov 12, 2018

Also what can be seen on this gif is another bug(I don't believe that it's intended behaviour): scrolling the release notes with MMB held selects the text. Same effect as holding LMB over text and scroling with keyboard arrow keys. When VS Code doesn't have focus, selection still appears and MMB scrolling doesn't work. - Windows, idk about other OSes.

@mjbvz mjbvz added the help wanted Issues identified as good community contribution opportunities label Nov 12, 2018
@rrmistry
Copy link

I do not notice this issue anymore with January 2019 (version 1.31) release.

Version: 1.31.1 (system setup)
Commit: 1b8e830
Date: 2019-02-12T02:20:54.427Z
Electron: 3.1.2
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.17763

@mjbvz
Copy link
Collaborator

mjbvz commented Feb 25, 2019

@Steve-Fry Can you please verify this too?

It may have been fixed when we updated to electron 3

@mjbvz mjbvz added the info-needed Issue requires more information from poster label Feb 25, 2019
@Steve-Fry
Copy link
Author

Steve-Fry commented Mar 4, 2019 via email

@mjbvz mjbvz added this to the March 2019 milestone Mar 4, 2019
@mjbvz mjbvz closed this as completed Mar 4, 2019
@mjbvz mjbvz added bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels Mar 4, 2019
@alexr00 alexr00 added the verified Verification succeeded label Mar 27, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Apr 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug electron Issues and items related to Electron help wanted Issues identified as good community contribution opportunities upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded webview Webview issues
Projects
None yet
Development

No branches or pull requests

7 participants