-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[filesystem] Fixed file dialog opening folder #4868
[filesystem] Fixed file dialog opening folder #4868
Conversation
Signed-off-by: Federico Bozzini <federico.bozzini@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also align the ElectronFileDialogService
?
https://github.com/theia-ide/theia/blob/7d37b1245ab15f91ffdcf8606ffe7b5efe02ba2f/packages/filesystem/src/electron-browser/file-dialog/electron-file-dialog-service.ts#L43
Sorry, I see you already updated the Electron version as well 😄 |
packages/filesystem/src/browser/file-dialog/file-dialog-service.ts
Outdated
Show resolved
Hide resolved
|
Should we? The current behaviour seems different than what I would expect. At the moment if I click |
As far as I know it was done on purpose: #332 cc @svenefftinge @federicobozzini please sign your commits and ECA: https://accounts.eclipse.org/users/akosyakov/eca |
Signed-off-by: Federico Bozzini <federico.bozzini@gmail.com>
193c294
to
a002a95
Compare
I did it like that because you probably want to switch to a sibling workspace as this is how many devs organize multiple workspaces locally. |
Is this PR accepted? Can it be merged? |
A file dialog is opened in the folder used as a parameter.
Fixes #4862