Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allowing TMUX pane titles to persist #66

Merged
merged 1 commit into from
Aug 10, 2022
Merged

Conversation

akd5027
Copy link
Contributor

@akd5027 akd5027 commented Sep 24, 2021

When running within tmux, attempt the following at the prompt

```bash
> tmux set -g pane-border-status bottom
> tmux set -g allow-rename off
> tmux set -g pane-border-format "< #T >"
> tmux split-window

> tmux select-pane -T "New Title"
```    

Prior to this change, the pureline_pre function is constantly setting the gui window title. This is the same value that TMUX uses to determine the pane title. This 'resetting' of the xterm title overwrites the tmux pane title. For this reason, and only within properly branded TMUX sessions, we do not reset the window title.

When running within tmux, attempt the following at the prompt
    
    ```bash
    > tmux set -g pane-border-status bottom
    > tmux set -g allow-rename off
    > tmux set -g pane-border-format "< #T >"
    > tmux split-window
    
    > tmux select-pane -T "New Title"
    ```    
Prior to this change, the pureline_pre function is constantly setting the gui window title.  This is the same value that TMUX uses to determine the pane title.  This 'resetting' of the xterm title overwrites the tmux pane title.  For this reason, and only within properly branded TMUX sessions, we do not reset the window title.
@akd5027
Copy link
Contributor Author

akd5027 commented Sep 24, 2021

Prior to this change, you would need to run a tight loop to even see the title changing:

while tmux select-pane -T "Testing"; do :; done

CyrilOtheninGirard added a commit to CyrilOtheninGirard/pureline that referenced this pull request Aug 9, 2022
This fix also allows users to prevent pureline from updating the GUI window title : they just have to unset PL_TITLEBAR in their config file.

Pull request chris-marsh#66 "Allowing TMUX pane titles to persist" describes one case where it is desirable.
CyrilOtheninGirard added a commit to CyrilOtheninGirard/pureline that referenced this pull request Aug 9, 2022
This fix also allows users to prevent pureline from updating the GUI window title : they just have to unset PL_TITLEBAR in their config file.

Pull request chris-marsh#66 "Allowing TMUX pane titles to persist" describes one case where it is desirable.
@CyrilOtheninGirard
Copy link
Contributor

CyrilOtheninGirard commented Aug 9, 2022

If #82 is merged, then you will have a simple way to prevent pureline from updating the GUI window title when using TMUX.

Just add this line to your pureline configuration file :

# If using tmux, allow pane titles to persist
[ -n "$TMUX" ] && unset PL_TITLEBAR

CyrilOtheninGirard added a commit to CyrilOtheninGirard/pureline that referenced this pull request Aug 9, 2022
This fix also allows users to prevent pureline from updating the GUI window title : they just have to unset PL_TITLEBAR in their config file.

Pull request chris-marsh#66 "Allowing TMUX pane titles to persist" describes one case where it is desirable.
CyrilOtheninGirard added a commit to CyrilOtheninGirard/pureline that referenced this pull request Aug 9, 2022
This fix also allows users to prevent pureline from updating the GUI window title : they just have to unset PL_TITLEBAR in their config file.

Pull request chris-marsh#66 "Allowing TMUX pane titles to persist" describes one case where it is desirable.
CyrilOtheninGirard added a commit to CyrilOtheninGirard/pureline that referenced this pull request Aug 9, 2022
This fix also offers a simple way to prevent pureline from updating the GUI window title:
the user just has to unset PL_TITLEBAR in its config file.

Pull request chris-marsh#66 "Allowing TMUX pane titles to persist" describes one case where it is desirable.
@CyrilOtheninGirard
Copy link
Contributor

(sorry for the mess, it's my first time with GIT, and thus with github and pull requests)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants