Close buffer from buffer explorer #59

Closed
wants to merge 2 commits into
from

Projects

None yet

5 participants

@darookee

I added the mapping of C-d to delete a buffer from within the buffer explorer. I'm not sure how stable or secure this is, as I'm not the vim-script or ruby guy... But it works without any errors so far...

@grota
Contributor
grota commented Jun 25, 2012

@sjbach what are your thoughts about this feature?
disclaimer: I did not test this, I only think that a buffer-delete feature would be cool in lusty.

@sjbach
Owner
sjbach commented Jun 25, 2012

Having functionality to remove buffers is dicey; you want to be sure you get it right, or the user is very sad. I haven't had time to fully review the patch and its effect, and I'm not encouraged by "I'm not the vim-script or ruby guy... But it works without any errors so far...".

@darookee

I agree with @sjbach, it should work as the user expects. It worked quite well for me and so I thought I should share...

@alovak
Contributor
alovak commented Aug 21, 2012

@darookee it seems to me that we try to add similar functionality to lusty.
do you left buffer explorer open after buffer was deleted?

@sjbach it means that this function is valuable for vimers :)

@darookee

@alovak the lusty buffer closes after deleting the selected buffer. As far as I can see our code is nearly the same, but yours looks much cleaner ;-)

@michalfapso

@darookee Thanks a lot for the patch! I was really missing this feature in lusty.

@sjbach
Owner
sjbach commented Mar 15, 2017

I think this feature was implemented in another pull request. Closing - thanks for the work nonetheless.

@sjbach sjbach closed this Mar 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment