[BUG] Focus next doesn't work about pytermgui

[bczsalba]

Should be fixed. Thank you for the report!

[jonnieey]

Just tested, doesn't work as expected. If you only have two windows it doesn't work. Also if you use many windows eg. sand box/layouts.py it doesn't work as it should.

[bczsalba]

Yup, just remembered figuring this one out ages ago. Basically the problem is this:

# Windows shown as indices for simplicity's sake
self._windows = [0, 1, 2, 3]
self.focus_next()
self._windows = [1, 0, 2, 3]
self.focus_next()
self._windows = [0, 1, 2, 3]

Not sure how well that explains it. The point is, when a window is focused it is pushed a slot down the "stack", so when the next window is focused over and over it ends up alternating between 2 windows. The way this was resolved back in the day was by tracking the currently focused window's index as an instance attribute, and referring to it in this call. I really didn't like that solution, which is why it no longer exists.

I'll try to come up with something that fixes it.

[jonnieey]

Sure, thanks.

[bczsalba]

Sorry about the wait, not sure how I forgot about this one.