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 option to disable collapsing of directory names #558

Merged
merged 3 commits into from
Sep 8, 2016
Merged

Add option to disable collapsing of directory names #558

merged 3 commits into from
Sep 8, 2016

Conversation

juanibiapina
Copy link
Contributor

This pull request adds the following option: NERDTreeCascadeSingleChildDir,
with default value of 1 (in order to keep the current behavior).

It can be disabled to prevent directory names from being collapsed into the same
line.

It doesn't affect the behavior of NERDTreeCascadeOpenSingleChildDir, because
it only changes how directories are displayed, not opened. Both options can be
used together in any combination.

After making the changes I realized "cascading" can generally be replaced with
"collapse", except when referring to "cascade open". I decided not to make this
change on this pull request.

@juanibiapina juanibiapina changed the title Add option to disable collapsing of diretory names Add option to disable collapsing of directory names Mar 5, 2016
@juanibiapina
Copy link
Contributor Author

@scrooloose

@shannonmoeller
Copy link

@scrooloose Please merge this, or implement similar.

#76

@nmhnmh
Copy link

nmhnmh commented Sep 8, 2016

@scrooloose please merge this, I really need this

@hookdump
Copy link

Thank god this option exists. How is "1" the default value for this? The option does not work. I cannot open directories that have been collapsed.

@lifecrisis
Copy link
Contributor

@hookdump, you don't need to open directories that have been collapsed because there is nothing else in them. This is a very useful feature in a number of programming contexts. It still has bugs, but collapsing directories by default is not one of them.

@juanibiapina
Copy link
Contributor Author

@hookdump The default value is 1 to keep backwards compatibility.

@shannonmoeller
Copy link

@PhilRunninger Thank you for merging this! Hadn't noticed until @hookdump commented. Soooo much better.

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

Successfully merging this pull request may close these issues.

6 participants