toggle explorer instead of opening a new one

[N-R-K]

This makes it so running :NnnExplorer while an explorer window is already open toggle/close it instead of opening a new explorer window.

Undocumented, also might need more testing (on both vim and neovim) before merge.

[jarun]

Thank you!

[N-R-K]

@jarun uhhh, the PR wasn't ready for merge yet 😅 . Wanted some more testing in case there's some weird quirk that happens on vim but not nvim and vice versa.

Welp, shouldn't be a big deal. If there's some quirk/problem, we can fix it on the go.

[jarun]

Use the label work in progress for this purpose.

[N-R-K]

Don't think I can add labels, which is why I put [WIP] on the PR title.

[jarun]

@mcchrish can you please add @N-R-K to the project?

[mcchrish]

Don't think I can add labels, which is why I put [WIP] on the PR title.

I believe you should be able to create a draft PR?

[N-R-K]

I believe you should be able to create a draft PR?

I'll try that next time. Anyways, since it's already merged don't think the label is needed. And it works fine for all "basic" usecases from what I've tested (both vim and nvim), so this should be fine.

[jarun]

@mcchrish in any case, please add @N-R-K to the project.

@N-R-K please confirm when you are added.

[N-R-K]

Thanks, I'm in.

[mcchrish]

@jarun @N-R-K you have merge privileges but let's make sure the one who merged tested the changes and confirms it's working as expected.

[mcchrish]

Probably don't need to initialize this immediately.

[N-R-K]

Yes, it's leftover from earlier state. I'll clean it up and document the changes on a separate PR.

[luukvbaal]

Closing the explorer pane by means other than running NnnExplorer will require you to run NnnExplorer twice to open it again with the way you implemented this I think(haven't tested). Which is why I actively check if the window is open in nnn.nvim.

Alternatively I suppose you could clear the tabpage var in the termopen() on_exit callback which might be cleaner now that I think of it.

[N-R-K]

@luukvbaal Yes, moving that in on_exit should fix the problem. I'll wait to see if there's any more problems before creating a new PR.