Skip to content
This repository has been archived by the owner on Jun 9, 2022. It is now read-only.

Open multiple paths in the same window as separate root folders #95

Closed
wants to merge 1 commit into from

Conversation

LordGaav
Copy link

@LordGaav LordGaav commented Mar 9, 2015

I changed openProject to open the given paths in the current window, by overwriting the currently opened folders. Before this change, if one would open multiple root folders and then save the project, loading the project would cause all paths to be loaded in separate windows. With my change, no new window is opened and all saved paths are properly loaded as root folders.

Unfortunately, I couldn't figure out how to preserve the devMode functionality of openProject. I also couldn't figure out how to open a new window and then set the saved project folders on that new window. This would make it more in line with the behaviour that is currently exhibited.

My Coffeescript / Atom dev knowledge is very limited, but maybe you could provide some pointers on how to achieve the two points I currently miss: devMode and opening a new window instead of replacing the current.

@Zireael07
Copy link

Please can this get merged ASAP? I often work with multiple root folders...

@danielbrodin
Copy link
Owner

First off it breaks functionality so it can't be merged. Second thing it does is break the current project you are in. It takes the current project and replaces its directories with a pair of new directories, so next time you would open the project without the project manager, the wrong folders would show.

So unfortunately this is not the correct way to do it.

@LordGaav
Copy link
Author

I know, my original intention was to open a new window with the given set op folders. But I'm going to need some pointers on how to do that in Atom.

The "open multiple folders" functionality is broken now anyways, because all folders are opened in separate windows, even if the saved project had all folders in one window.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants