omerxx / tmux-sessionx

A Tmux session manager, with preview, fuzzy finding, and MORE
GNU General Public License v3.0
718 stars 63 forks source link

Current session not in the list - sometimes #8

Closed stepanzak closed 9 months ago

stepanzak commented 10 months ago

When I first enter the sessionx menu with \+O, there are all sessions except the current one. When I select any of them and press \ and rename it to anything (or just press enter to leave it the same), the current session also appears on the list. This happens only when I have more sessions than one.

System info: OS: EndeavourOS Linux (Arch based) Shell: ZSH Latest version of everything

Screenshots:

![image](https://github.com/omerxx/tmux-sessionx/assets/99094176/740aa6ef-090a-4a24-8fb5-e7da52653cab) ![image](https://github.com/omerxx/tmux-sessionx/assets/99094176/031b97e5-2379-43bb-a98d-b2fc9bef1cc5) ![Screenshot_20231210_211816](https://github.com/omerxx/tmux-sessionx/assets/99094176/5e42a634-a40c-4ce0-ac92-f6c025ec9bdf)

omerxx commented 9 months ago

Yep, actually that was done intentionally to filter out the current session on first glance for the quick switch which is 90% of my use of the plugin. After rename I bring it back assuming the user is running organization tasks and would want to rename the current session as well.

@stepanzak That said, I understand it's not a consistent behaviour and feels awkward. WDYT?

stepanzak commented 9 months ago

Hmm, it actually makes lot of sense when you say it. Showing the current session is AFAIK useful only for renaming and for that you usually use \+$. The problem is that it's confusing. I can think of a few options:

I'm not a UX expert or anything though, this is just what was on top of my head

omerxx commented 9 months ago

@stepanzak Yep I thought of all of these... I think I'll go with the readme option + a configureable option to have it on by default