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

Add "Open Folder..." to macOS and rename "Open Workspace..." #131793

Closed
isidorn opened this issue Aug 27, 2021 · 12 comments
Closed

Add "Open Folder..." to macOS and rename "Open Workspace..." #131793

isidorn opened this issue Aug 27, 2021 · 12 comments
Assignees
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders menus Menu items and widget issues on-release-notes Issue/pull request mentioned in release notes verification-needed Verification of issue is requested verified Verification succeeded

Comments

@isidorn
Copy link
Contributor

isidorn commented Aug 27, 2021

I was just in a user study with @stevencl and I think we have a problem that IDE users have a concept of workspace which is not the VS Code Workspace.

So what happens -> users click Open Workspace and actually want to open the whole folder, however they can not since you expect a .code-workspace file.

Due to that I suggest (if technically possible) that the Open Workspace button allows to open .code-workspace file but also any folder.

This is especially confusing on mac, since the Open.. button opens everything including folders, so users go for Open Workspace and end up being puzzled.

@isidorn isidorn added the under-discussion Issue is under discussion for relevance, priority, approach label Aug 27, 2021
@bpasero
Copy link
Member

bpasero commented Aug 27, 2021

@isidorn this would only be possible on macOS, not Windows nor Linux. It might break muscle memory for existing users that e.g. double click on a folder in the dialog to open it and to find *.code-workspace files.

@isidorn
Copy link
Contributor Author

isidorn commented Aug 27, 2021

@bpasero I think it would be a great fit for macOS, since there we have the other confusion that there is no Open Folder button, just Open. And this is exactly what happened in the user study (user wanted to open a folder, and closest to his need he found the Open Workspace button).

I do not understand muscle memory breakage sorry, can you clarify please?

@bpasero
Copy link
Member

bpasero commented Aug 27, 2021

@isidorn see this example, I double click a folder to open it. With your proposal that would not be possible anymore:

recording (2)

@isidorn
Copy link
Contributor Author

isidorn commented Aug 30, 2021

@bpasero aha got it, thanks! Good point.
Maybe we bring it up quickly in a UX meeting? Or we try it in Insiders and listen to negative feedback? What do you think?

@bpasero
Copy link
Member

bpasero commented Aug 30, 2021

UX meeting sounds fine to me, since I have my doubts about it I would not push it to insiders without a discussion first.

@bpasero bpasero added this to the September 2021 milestone Aug 30, 2021
@bpasero bpasero added the ux User experience issues label Aug 30, 2021
@bpasero
Copy link
Member

bpasero commented Sep 1, 2021

@isidorn spend a bit more time thinking about this and feel we should try to be consistent across OS, so what about:

  • we add "Open Folder..." on macOS too (which then puts us in the same category as the other OS)
  • we rename "Open Workspace..." to "Open Multi-Root Workspace..." (I think I got similar feedback from Consistency with the use of "workspace"  #77718 that the name is a bit misleading)

I would still not change "Open..." on macOS because I think it works OK today and changing it would probably confuse existing users.

@isidorn
Copy link
Contributor Author

isidorn commented Sep 1, 2021

Love this suggestion 💯
Open Workspace is too general, so Open Multi-Root Workspace will hit the spot. Open Folder.. also makes good sense to be more explicit.

Should we still bring this up in the UX today or should I remove it from the agenda?

@bpasero
Copy link
Member

bpasero commented Sep 1, 2021

Since we do not have many items, feel free to bring it up.

@gjsjohnmurray
Copy link
Contributor

How about Open Workspace from File... instead? Not all such workspaces have multiple roots.

@bpasero
Copy link
Member

bpasero commented Sep 1, 2021

Yeah I like that, wasn't so happy with the wording I chose.

@bpasero bpasero closed this as completed in 218fc09 Sep 1, 2021
@bpasero bpasero added feature-request Request for new features or functionality menus Menu items and widget issues and removed ux User experience issues under-discussion Issue is under discussion for relevance, priority, approach labels Sep 1, 2021
@bpasero bpasero changed the title Open Workspace should allow to open folder Add "Open Folder..." to macOS and rename "Open Workspace..." Sep 1, 2021
@bpasero
Copy link
Member

bpasero commented Sep 1, 2021

image

Also in Welcome:

image

@bpasero bpasero added the verification-needed Verification of issue is requested label Sep 17, 2021
@bpasero
Copy link
Member

bpasero commented Sep 24, 2021

Verification:

  • where previously we would have "Open Workspace" it is now "Open Workspace from File"
  • macOS: there is a "Open Folder" entry in the file menu and command palette that only allows to open folders

@rzhao271 rzhao271 added the verified Verification succeeded label Sep 28, 2021
@bpasero bpasero added the on-release-notes Issue/pull request mentioned in release notes label Sep 30, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Oct 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders menus Menu items and widget issues on-release-notes Issue/pull request mentioned in release notes verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants
@bpasero @isidorn @gjsjohnmurray @rzhao271 and others