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

Make behaviour of "Open Workspace/Folder" consistent with VS Code #12537

Merged
merged 2 commits into from
May 24, 2023

Conversation

tsmaeder
Copy link
Contributor

What it does

The PR renames the "Open Workspace" file menu to "Open Workspace from File" and only allows selection of files in the resulting dialog on all platformas and browser/electron.

Fixes #12534

Contributed on behalf of STMicroelectronics

How to test

Use the "Open Workspace from File..." menu and make sure the behavior is as described above acrosss all platforms and electron/browser.

Review checklist

Reminder for reviewers

Copy link
Contributor

@alvsan09 alvsan09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested the change and it works as expected 👍
i.e., The behavior in Electron and Browser are now consistent and equivalent to vscode

A minor inline remarks were raised.

Fixes eclipse-theia#12534

Contributed on behalf of STMicroelectronics

Signed-off-by: Thomas Mäder <t.s.maeder@gmail.com>
@tsmaeder tsmaeder requested a review from alvsan09 May 22, 2023 14:06
Signed-off-by: Thomas Mäder <t.s.maeder@gmail.com>
Copy link
Contributor

@alvsan09 alvsan09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested the functionality in the browser as well as in electron and the behavior is as expected 👍

The code LGTM !!

@tsmaeder tsmaeder merged commit 8a5750d into eclipse-theia:master May 24, 2023
@vince-fugnitto vince-fugnitto added this to the 1.38.0 milestone May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inconsisten Behaviour of "Open Workspace" on Electron vs. Browser
3 participants