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

Quickpick doesn't work when items are updated #6626

Open
DanTup opened this issue Nov 25, 2019 · 4 comments
Open

Quickpick doesn't work when items are updated #6626

DanTup opened this issue Nov 25, 2019 · 4 comments
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help quick-open issues related to the quick-open vscode issues related to VSCode compatibility

Comments

@DanTup
Copy link
Contributor

DanTup commented Nov 25, 2019

I'm trying to test the asExternalUri support using the instructions from @akosyakov at #6465 (comment) (though I'm using the master branch since that PR is now merged).

When the Dart extension opens the device picker (either by trying to launch an app or clicking on the device label in the status bar) the selector appears but it's impossible to select an item (either with the keyboard or the mouse):

Screenshot 2019-11-25 at 3 51 16 pm

@vince-fugnitto vince-fugnitto added the quick-open issues related to the quick-open label Nov 25, 2019
@akosyakov
Copy link
Member

@DanTup Could I try it somehow?

@akosyakov akosyakov added bug bugs found in the application vscode issues related to VSCode compatibility labels Nov 25, 2019
@DanTup
Copy link
Contributor Author

DanTup commented Nov 26, 2019

Sorry, I should've set something up :)

I tried forking and setting up a branch to more easily repro here, but it seems to stall when starting up

https://github.com/DanTup/theia/tree/picker-test

Screenshot 2019-11-26 at 10 07 59 am

It's been like that for quite a while (I tried refreshing the page, but it didn't change). It hasn't bound to port 3000 yet so I can't open it to test.

Scratch that - it just finished - I'll post repro notes back shortly...

@DanTup
Copy link
Contributor Author

DanTup commented Nov 26, 2019

Ok, the Dart version didn't repro (I'm not sure what the difference is yet). But here's a Flutter one:

https://gitpod.io/#https://github.com/DanTup/theia/tree/picker-test

If you run that, then open the app on port 3000, open my_flutter_test/lib/main.dart and then click the "Web Server" label in the status bar, it'll show the device picker:

Screenshot 2019-11-26 at 11 03 44 am

However, you can't select either of these items. I don't know if it's related, but I have this in the dev console:

Screenshot 2019-11-26 at 11 04 19 am

@DanTup
Copy link
Contributor Author

DanTup commented Nov 26, 2019

Oh ok, I know what's different - for Flutter we update the list after it's been shown (because some work is async, we may add new items to the list). We do this by re-assigning the items property:

https://github.com/Dart-Code/Dart-Code/blob/cf79ce75cd616e6cf9c43bd866d76c9ff6c6bea1/src/shared/vscode/device_manager.ts#L93-L123

This ability was added in VS Code in v1.26:

https://code.visualstudio.com/updates/v1_26#_quickinput-api

(possibly the same time multi-select was added - #5673).

@DanTup DanTup changed the title Quickpick doesn't work Quickpick doesn't work when items are updated Nov 26, 2019
@akosyakov akosyakov added the help wanted issues meant to be picked up, require help label Nov 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help quick-open issues related to the quick-open vscode issues related to VSCode compatibility
Projects
None yet
Development

No branches or pull requests

3 participants