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 an option to open NERDTree in current window #839

Closed
wants to merge 1 commit into from

Conversation

zefei
Copy link

@zefei zefei commented May 15, 2018

In a multi-window workflow, it's often desirable to make NERDTree to
work in the current window instead of its side window. See issue #244
for brief discussion.

Currently it's possible to use :e <dir> to achieve this for
:NERDTree command; but there are no workarounds for other functions
like opening bookmark, toggling nerdtree, finding current file, etc.

This commit adds an option to render NERDTree in current window instead
of side window, and modifies all relevant commands to respect the
option.

In a multi-window workflow, it's often desirable to make NERDTree to
work in the current window instead of its side window. See issue preservim#244
for brief discussion.

Currently it's possible to use `:e <dir>` to achieve this for
`:NERDTree` command; but there are no workarounds for other functions
like opening bookmark, toggling nerdtree, finding current file, etc.

This commit adds an option to render NERDTree in current window instead
of side window, and modifies all relevant commands to respect the
option.
@lifecrisis
Copy link
Contributor

This is not a pull request I want to merge.

We're trying not to add any more global options. So, I suggest that you add this functionality by extending the NERDTree with a plugin for your own private use. This PR adds a lot of code, which is no good for an already bloated project.

I don't mean to be harsh, but this is not the direction in which I want the NERDTree to go. Feel free to submit PRs in the future. Bug-fixes are especially welcome at this time.

@lifecrisis lifecrisis closed this May 15, 2018
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.

2 participants