omerxx / tmux-sessionx

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

List tmuxinator projects with `--newline` #83

Closed cemcatik closed 2 months ago

cemcatik commented 2 months ago

Given multiple tmuxinator projects, when the list is not long enough, the tmuxinator list command lists the projects in columns. This causes the projects to not render as list.

tmuxinator accepts --newline argument to list command. See https://github.com/tmuxinator/tmuxinator/blob/master/lib/tmuxinator/cli.rb#L397-L399. This PR uses the --newline argument so all tmuxinator projects list properly.

$ tmuxinator new a
...
$ tmuxinator new b
...
$ tmuxinator list
tmuxinator projects:
a    b

In tmuxinator view this list shows up as

> a    b
$ tmuxinator list --newline
tmuxinator projects:
a
b

With this PR, in tmuxinator view this would show up as separate projects.

cemcatik commented 2 months ago

Should this also need to be updated?

https://github.com/omerxx/tmux-sessionx/blob/8c4ec54a3f7c1e42e0c4c4bcd4bb6853577581b4/scripts/tmuxinator.sh#L3-L5

omerxx commented 2 months ago

LGTM

cemcatik commented 2 months ago

For consistency I'll update is_known_tmuxinator_template to also pass the --newline flag.

cemcatik commented 2 months ago

For consistency I'll update is_known_tmuxinator_template to also pass the --newline flag.

Done