fix(renderer): after focusing a node, such node was always focused

[georgeguimaraes]

Sorry for the bug. This should fix it.

Do you prefer to have the code without the comment?

Fixes #1279

[cseickel]

Thanks for the quick fix. No need to remove the comment, you can never have too many comments.

[cseickel]

Darn. I got so used to bypassing one flaky test that I let this through without noticing that it broke more tests. I'm removing that flaky test now and I'll take a look at the newly broken tests:

:Neotree show toggle should retain the focused node on next show
:Neotree focus toggle should retain the focused node on next focus

[georgeguimaraes]

I'll take a look too.

[georgeguimaraes]

Yeah, when we call manager.close, I'm not saving the position in this new way. So the cursor is always at the first line.
I'm experimenting adding renderer.position.save on some other points of code, but still no success yet. I'm still trying though, just wanted to give a heads up.

[georgeguimaraes]

PR with fix available at #1288. Tests passing!