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

Make it Possible to Add Child Folders as Root Folders #404

Closed
craighooghiem opened this issue Mar 12, 2015 · 6 comments
Closed

Make it Possible to Add Child Folders as Root Folders #404

craighooghiem opened this issue Mar 12, 2015 · 6 comments
Assignees

Comments

@craighooghiem
Copy link

Now that we can add root folders to the tree-view (#63) it would be great if we could make a small adjustment to make it function even more like alternative editors do.

Currently I can add additional root folders to a project to allow for a more organized tree view, but I cannot add a root folder if it is a child of an existing root folder.

Use Case: Working on a project with a library or vendor folder that goes multiple folders deep. I would like to be able to organize the tree view better to make it easier to navigate these folders.

screen shot 2015-03-12 at 7 37 52 am

screen shot 2015-03-12 at 7 38 17 am

@fpupor
Copy link

fpupor commented Mar 13, 2015

I think that this function is really necessary, I use a lot in other editors

@timodwhit
Copy link

👍

@mathiasvr
Copy link

This can be achieved by simply removing the if directory.contains(projectPath) condition at line 211 in project.coffee. But maybe it should be indicated when child folders are added like this.

@ghost
Copy link

ghost commented Apr 6, 2015

I agree, wholeheartedly!

The way the comment above project.coffee Line 193 is written makes me think that not duplicating entries in project.rootDirectories was an afterthought, and the main idea is to prevent users from doing exactly this:

      # Apparently a Directory does not believe it can contain itself, so we
      # must also check whether the paths match.
      return if directory.contains(projectPath) or directory.getPath() is projectPath

It's a totally valid use case though. I mean, I'd like the flexibility, anyway.

@marjakm
Copy link

marjakm commented Jul 8, 2015

Would like to have this feature

@maxbrunsfeld
Copy link
Contributor

Hey everyone, sorry for the wait. This will be fixed in Atom 1.0.8.

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

No branches or pull requests

7 participants