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

TreeItem on click commands don't work #5744

Closed
tetchel opened this issue Jul 18, 2019 · 7 comments
Closed

TreeItem on click commands don't work #5744

tetchel opened this issue Jul 18, 2019 · 7 comments
Labels
bug bugs found in the application vscode issues related to VSCode compatibility

Comments

@tetchel
Copy link
Contributor

tetchel commented Jul 18, 2019

With #5661 in, commands no longer break the tree, but the commands do not work (they never execute, so I assume there is just no listener).

In VS Code, single-clicking a tree item runs the command tied to it, eg opening a file in the Explorer view.
This can be configured to be double-click instead with the setting workbench.list.openMode. (see microsoft/vscode#39601)

TreeItem onclick commands do not appear to have any effect in Theia.

@vince-fugnitto
Copy link
Member

@tetchel which command were you attempting to execute with the tree?

@akosyakov akosyakov added bug bugs found in the application vscode issues related to VSCode compatibility labels Jul 18, 2019
@tetchel
Copy link
Contributor Author

tetchel commented Jul 18, 2019

A custom one from my extension

Edit: also revealInExplorer, not sure if Theia has that builtin.

@vince-fugnitto
Copy link
Member

Edit: also revealInExplorer, not sure if Theia has that builtin.

We do have the command Reveal in Explorer
https://github.com/theia-ide/theia/blob/d4a15ad515666ead71556faf04c226d9157d3a33/packages/navigator/src/browser/navigator-contribution.ts#L38

@tetchel
Copy link
Contributor Author

tetchel commented Jul 18, 2019

hmm, that is a different command ID than vs code (again, revealInExplorer), perhaps that should be a separate issue.

@vince-fugnitto
Copy link
Member

hmm, that is a different command ID than vs code (again, revealInExplorer), perhaps that should be a separate issue.

I believe it will be, it's part of the #5590 change :)

@akosyakov
Copy link
Member

akosyakov commented Aug 23, 2019

@tetchel Can it be closed by #5961? Could you verify please?

@tetchel
Copy link
Contributor Author

tetchel commented Sep 19, 2019

Indeed this is fixed in 0.10, thanks guys

@tetchel tetchel closed this as completed Sep 19, 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 vscode issues related to VSCode compatibility
Projects
None yet
Development

No branches or pull requests

3 participants