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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quick pick shows wrong selected items #91661

Closed
sandy081 opened this issue Feb 27, 2020 · 3 comments 路 Fixed by #91741
Closed

Quick pick shows wrong selected items #91661

sandy081 opened this issue Feb 27, 2020 · 3 comments 路 Fixed by #91741
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug candidate Issue identified as probable candidate for fixing in the next release quick-pick Quick-pick widget issues settings-sync verified Verification succeeded
Milestone

Comments

@sandy081
Copy link
Member

Steps to Repro

  • Set up settings sync - Follow instructions here
  • Turn on Sync
  • From the gear icon, select Sync is on entry
  • Quick pick is shown. From the quick pick click on Sync: Configure entry
    馃悰 Quick pick with wrong selected items is shown

image

Expected:

image

Note: Expected is shown when you use keyboard to select Sync: Configure entry. It happens only when you use mouse and click.

@sandy081 sandy081 added bug Issue identified by VS Code Team member as probable bug quick-pick Quick-pick widget issues labels Feb 27, 2020
@chrmarti
Copy link
Contributor

@sandy081 There is a change event on the checkbox from the mouse up on the Sync: Configure entry. I don't have a fix in QuickInput currently, can we change

run(): any { return that.configureSyncOptions(); }
to:

run(): any { return new Promise(r => setTimeout(r)).then(() => that.configureSyncOptions()); }

That appears to let the mouse up pass without triggering the change event on the checkbox.

@sandy081 sandy081 added candidate Issue identified as probable candidate for fixing in the next release settings-sync labels Feb 28, 2020
@sandy081 sandy081 added this to the February 2020 milestone Feb 28, 2020
@sandy081
Copy link
Member Author

@chrmarti Is this a temp workaround and do you have plans to fix it in quick pick later?

@chrmarti
Copy link
Contributor

@sandy081 It鈥檚 certainly a workaround. I don鈥檛 have a proper fix yet.

This was referenced Feb 28, 2020
sandy081 added a commit that referenced this issue Feb 28, 2020
@roblourens roblourens added the verified Verification succeeded label Feb 28, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Apr 13, 2020
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 candidate Issue identified as probable candidate for fixing in the next release quick-pick Quick-pick widget issues settings-sync verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants