Add prefix key and pane sync indicators

jimeh / tmux-themepack

A pack of various Tmux themes.

1.67k stars 203 forks source link

Add prefix key and pane sync indicators #34

Open jimeh opened 4 years ago

jimeh commented 4 years ago

The indicators used can be customized:

For the prefix key indicator, set @themepack-prefix-key-on-format and @themepack-prefix-key-off-format options. The off format must be the same length as the on format to avoid weird behavior.
For the pane sync indicator, set @themepack-pane-sync-on-format and @themepack-pane-sync-off-format options. The off format must be the same length as the on format to avoid weird behavior.

To-Do

[ ] Fix duplicated and glitchy indicators when reloading tmux configuration.
[ ] Figure out if there's an easy way for users disable the indicators if they don't want them.

Gifs

Prefix Key Indicator

Basic theme:

basic-prefix-key-indicator

Powerline themes:

powerline-prefix-key-indicator

Pane Sync Indicator

Basic theme:

basic-pane-sync-indicator

Powerline themes:

powerline-pane-sync-indicator

lovesegfault commented 4 years ago

This looks awesome! Thank you!

ori955 commented 4 years ago

thank you so much! really like your work!

Blake-LeBlanc commented 4 years ago

Hey @jimeh ! Maybe I'm blind as a bat, but I can't seem to find where the @themepack-prefix-key-on-format and @themepack-prefix-key-off-format settings are located... Should I be looking for them within the theme I'm using? (for example, within the tmux-tmux/plugins/themepack/powerline/default/cyan.tmuxtheme file?)

jimeh commented 4 years ago

@Blake-LeBlanc Yeah, they should be defined within the theme file you're using. However make sure you're using the branch for this PR instead of master.

I've been meaning to find time for some final cleanup and added flexibility for this PR before I merge it, and so far it hasn't happened.

Blake-LeBlanc commented 4 years ago

@Blake-LeBlanc Yeah, they should be defined within the theme file you're using. However make sure you're using the branch for this PR instead of master.

I've been meaning to find time for some final cleanup and added flexibility for this PR before I merge it, and so far it hasn't happened.

Ahh, got it with the PR branch, thanks!

yeslsudarshan commented 4 years ago

Hey @jimeh! Let me first say, this is a great set of themes and this enhancement makes it even better. Thanks a lot for sharing this.

I tried this PR and things work great for the most part.

Only thing I noticed is that the status line gets messed up every time I source the tmux.conf and keeps adding separators on the Left and Right side.

Clean :

After sourcing .tmux.conf, notice the '>' separators:

And all the elements now indicate prefix key press:

Also, more separators get added every time tmux.conf is sourced:

jimeh commented 4 years ago

@yeslsudarshan thanks for bringing that to my attention. I hadn't noticed it myself, as it turns out I don't reload my tmux config very often.