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 second folder #30608

Closed
Tekbr opened this issue Jul 13, 2017 · 12 comments
Closed

Add second folder #30608

Tekbr opened this issue Jul 13, 2017 · 12 comments
Assignees
Labels
under-discussion Issue is under discussion for relevance, priority, approach workbench-multiroot Multi-root (multiple folders) issues
Milestone

Comments

@Tekbr
Copy link

Tekbr commented Jul 13, 2017

  • VSCode Version: 1.15.0 Insiders (56a5bf1, Date 2017-07-13T06:29:54.260Z)
  • OS Version: Windows 10

-- Sorry for the English, I used Google Translator --

Sorry for the question but ..

Question 1- But why am I forced to create a work area?

I simply want to add a second folder, without having to create a specific workspace.

Also when I create the work area it changes the color of the Status bar. (I do not think that would ask it to happen).

This window should have a third option, "Add only the folder".

workspace

That is, the options would be: "New Workspace", "Add only the folder", "Cancel".

Question 2 - What's more, why the "Close Folder" option changed "Close Workspace", even though a Workspace has not yet been created? The menu should change only when a workspace was created, right?

menunew

If you need to create a separate issue, please move one of them.

Reproduces without extensions: Yes

@bpasero
Copy link
Member

bpasero commented Jul 13, 2017

@Tekbr thanks for your feedback, this UI is very young and might change until the final bits are out. We want to make workspaces an explicit concept for multi root. You cannot have multiple folders in the classical single-folder views currently. Creating a workspace will bring you into a new experience (hence the status bar change).

I know this is different from how it works in stable currently, but we had lengthy design discussions around the flaws of the model in stable and decided to go with this approach.

We still think that you are in a workspace context for both single root and multi root, hence the action changed to "Close Workspace". You already see today that you are in a workspace context if you open a folder and open settings:

image

@bpasero bpasero added the under-discussion Issue is under discussion for relevance, priority, approach label Jul 13, 2017
@bpasero bpasero added this to the July 2017 milestone Jul 13, 2017
@bpasero
Copy link
Member

bpasero commented Jul 13, 2017

@Tekbr see #396 (comment)

@Tekbr
Copy link
Author

Tekbr commented Jul 13, 2017

-- Sorry for the English, I used Google Translator --

@bpasero Thanks for the quick response, for the attention and patience. Yes I know it's at the beginning, but I found it a bit strange. But I'll wait for the news (said by you in the #396 (comment))


-- My thought, perhaps the community has another vision --

I think a Workspace should be a project that has several folders. But sometimes we are upgrading the old project to a new project (or querying a similar project folder), and we want to add the old project folder, but without the need to bind.


Another note (which will probably change in the future): Unsaved workspace should not be placed in Recent Open until it is saved. By clicking "Close Workspace" and if it is not saved, it would be interesting to ask if you would like to save. If the user clicks No, it should not be placed in the menu.

Why that? Because the menu will be very polluted. In addition, you can not differentiate between "Untitled Workspace".

@bpasero
Copy link
Member

bpasero commented Jul 13, 2017

Yeah we think about asking you to save a untitled workspace once you close the window. Good point about the fact that untitled workspaces should not show up in the recent history 👍

@Tekbr
Copy link
Author

Tekbr commented Jul 13, 2017

-- Sorry for the English, I used Google Translator --

Yeah we think about asking you to save a untitled workspace once you close the window.

@bpasero I think the question should only remain when closing the Workspace, not when closing the window.

If I open another folder with VSCode (right button> Open With VSCode) - it can not be running - and if in the last session it was open an Untitled Workspce, maybe an alert asking if: I want to save the last Workspace before opening The new folder or discard the old Workspace and open the new folder or open last session before (and discard the current folder.)

@bpasero
Copy link
Member

bpasero commented Jul 13, 2017

@Tekbr our current thinking is to ask as soon as you close the window, otherwise we would need to introduce yet another way of dismissing a "Untitled Workspace".

@Tekbr
Copy link
Author

Tekbr commented Jul 13, 2017

-- Sorry for the English, I used Google Translator --

@bpasero I think it is best to discard only when we click on "Close Workspace", or the situation described above.

Recalling that within a Workspace there may be unsaved file and Untitled file. In this case it would be good to alert that modified and / or unsaved files (Untitled) could be lost. Or is there no such possibility? Once a workspace is saved, does it matter all the data (including files are saved)?

Or am I saying something silly? Sorry for anything...

@bpasero
Copy link
Member

bpasero commented Jul 14, 2017

@Tekbr sure, dirty files would also cause us to ask what to do (save), I would not change that behaviour.

If we decide to not cleanup an untitled workspace when you are not closing the window, we would still need to show all untitled workspaces in the "File > Open Recent" menu, how do you otherwise get back to them to close them? And we would need even more UI to dismiss an untitled workspace without opening it to manage them. I find that a lot of UI which can be avoided by just asking you to save the workspace when you close the window?

@Tekbr
Copy link
Author

Tekbr commented Jul 14, 2017

-- Sorry for the English, I used Google Translator --

@bpasero By what I understand (and according to Google Translate) ...

If we decide to not cleanup an untitled workspace when you are not closing the window, we would still need to show all untitled workspaces in the "File > Open Recent" menu, how do you otherwise get back to them to close them?

(I do not know if it's possible) - thought

Currently if I close the VSCode with a folder, and when I open it again it returns no was it, correct?

If you close the VSCode with an untitled workspace when opening the VSCode again it should open as closed, that is, with the untitled workspace.

If it detects that a new folder has been opened (other than the workspace) it should say 'I want to save the last workspace before opening the new folder' or if 'I want to discard the workspace and open the new folder' or 'go back to the last workspace And discard the current folder '. (As I said earlier)

Or option would be:

Limit to 2 or 3 untitled workspace, when the next one is created (the 4 untittled) would ask what you want to save or warn that the oldest will be deleted and you want to continue.

Showing next to the name untitled the date / time it was created (DD / MM / YYYY - HH: MM: SS) in File recent

I find that a lot of UI which can be avoided by just asking you to save the workspace when you close the window?

In this case you are obliged to save something that is not definitive that has no bond, but that I will use again in some mometo of the day or for some days.

Another idea (even to get more organized) would be to create "File> Open Recent Workspace"

@bpasero
Copy link
Member

bpasero commented Jul 15, 2017

@Tekbr it depends on the OS: On Windows and Linux if you close the last window, we close VS Code and in that case we would not ask to save the workspace and simply restore it the next time you open. However, on macOS, closing the last Window does not quit the application so we would ask to save the workspace.

This feature should work in the same way as "hot exit" works for files today imho.

Good idea about showing the date for untitled workspaces. However, I am not even convinced yet that untitled workspaces should show up under the "Files > Open Recent" category...

@bpasero
Copy link
Member

bpasero commented Jul 17, 2017

All these changes landed, I suggest to retest this and give it individual issues when seeing something. The smooth transition from 1 to 2 folders is something we have on our backlog.

@bpasero bpasero closed this as completed Jul 17, 2017
@bpasero bpasero added the workbench-multiroot Multi-root (multiple folders) issues label Jul 17, 2017
@Tekbr
Copy link
Author

Tekbr commented Jul 17, 2017

-- Sorry for the English, I used Google Translator --

@bpasero Sorry for the delay...

However, on macOS, closing the last Window does not quit the application so we would ask to save the workspace.

Well ... I've never used MacOS so far.
But from what I understand (according to Google Translate), if I want to close the VSCode window, in MacOS does it keep it open, closing like it's a folder in Windows or Linux? Is that it? Strange ...

So I suggested reopening works on Windows and Linux, but it does not work on MacOS. Would that be it?

It may be complex, but why not do it for Windows and Linux, since it works well for both? Some things have already been done where it only works for Mac or better on Mac.

Good idea about showing the date for untitled workspaces.

As for the date / time do not forget to use the format according to OS settings.

All these changes landed, I suggest to retest this and give it individual issues when seeing something. The smooth transition from 1 to 2 folders is something we have on our backlog.

Ok I'll check out the latest version of the Insiders

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
under-discussion Issue is under discussion for relevance, priority, approach workbench-multiroot Multi-root (multiple folders) issues
Projects
None yet
Development

No branches or pull requests

2 participants