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

Git multi select and stage is broken #43787

Closed
octref opened this issue Feb 15, 2018 · 5 comments
Closed

Git multi select and stage is broken #43787

octref opened this issue Feb 15, 2018 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug scm General SCM compound issues verified Verification succeeded
Milestone

Comments

@octref
Copy link
Contributor

octref commented Feb 15, 2018

Issue Type

Bug

Description

This is working in 1.19.3 but not 1.20.1.

@isidorn Don't know if it's due to the multi-select update on Viewlets or some changes in git. Giving to you first anyway.

bug

VS Code Info

VS Code version: Code - Insiders 1.21.0-insider (70e020b, 2018-02-15T06:08:32.814Z)
OS version: Darwin x64 17.3.0

System Info
Item Value
CPUs Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)
Load (avg) 2, 2, 2
Memory (System) 16.00GB (0.66GB free)
Process Argv /Applications/Visual Studio Code - Insiders.app/Contents/MacOS/Electron -psn_0_5334294
Screen Reader no
VM 18%
Extensions (7)
Extension Author (truncated) Version
vsc-material-theme Equ 1.3.0
tslint eg2 1.0.27
prettier-vscode esb 1.1.3
vscode-slm mrm 1.1.0
PowerShell ms- 1.5.1
vetur oct 0.11.7
vim vsc 0.10.13

(1 theme extensions excluded)

Reproduces without extensions
@octref octref added the git GIT issues label Feb 15, 2018
@isidorn
Copy link
Contributor

isidorn commented Feb 16, 2018

Nope, this is not list specific as it works nicely in the open editors. Forwarding to @joaomoreno
I can repro

@isidorn isidorn assigned joaomoreno and unassigned isidorn Feb 16, 2018
@joaomoreno
Copy link
Member

@bpasero @isidorn I did not touch this code since a long time. Care to check what your refactorings might have done here?

@joaomoreno joaomoreno added this to the February 2018 milestone Feb 19, 2018
@joaomoreno joaomoreno added bug Issue identified by VS Code Team member as probable bug scm General SCM compound issues and removed git GIT issues labels Feb 19, 2018
@isidorn
Copy link
Contributor

isidorn commented Feb 19, 2018

@joaomoreno this issue is older than my change in the list so probably @bpasero changes broke this.
I can also investigate if you are out of cycles @bpasero

@bpasero
Copy link
Member

bpasero commented Feb 20, 2018

It actually only reproduces sporadically for me...

@bpasero
Copy link
Member

bpasero commented Feb 20, 2018

This was because of options.selectOnMouseDown: true, the right click would select and open the element under the mouse. I am now checking for right-click before doing that because that should not be a button to use for opening/selecting.

@sandy081 sandy081 added the verified Verification succeeded label Mar 1, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Apr 6, 2018
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 scm General SCM compound issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants