Conversation
There was a problem hiding this comment.
Pull request overview
Updates the tmux status bar configuration to display the current user and host on the right side of the status line.
Changes:
- Replace the existing
status-righttime display withuser@hostrendering.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
home/modules/tmux/files/tmux.conf
Outdated
| status_left2="#S #{?client_prefix,#[fg=colour31]#[bg=colour234]#[nobold],#[fg=colour254]#[bg=colour234]#[nobold]}" | ||
| set -qg status-left $status_left1' '$status_left2 | ||
| set -qg status-right '%H:%M #[fg=colour16,bg=colour254]' | ||
| set -qg status-right ' #[fg=colour244]#(whoami)@#h #[fg=colour16,bg=colour254]' |
There was a problem hiding this comment.
#(whoami) runs an external shell command and will be executed on every status redraw (status-interval is 2s in this file), which adds unnecessary process spawn overhead. Prefer a built-in tmux format (if available) or a value computed once (e.g., via an environment/user option) to avoid running whoami repeatedly.
| set -qg status-right ' #[fg=colour244]#(whoami)@#h #[fg=colour16,bg=colour254]' | |
| set -g @status_user "$USER" | |
| set -qg status-right ' #[fg=colour244]#{@status_user}@#h #[fg=colour16,bg=colour254]' |
home/modules/tmux/files/tmux.conf
Outdated
| status_left2="#S #{?client_prefix,#[fg=colour31]#[bg=colour234]#[nobold],#[fg=colour254]#[bg=colour234]#[nobold]}" | ||
| set -qg status-left $status_left1' '$status_left2 | ||
| set -qg status-right '%H:%M #[fg=colour16,bg=colour254]' | ||
| set -qg status-right ' #[fg=colour244]#(whoami)@#h #[fg=colour16,bg=colour254]' |
There was a problem hiding this comment.
This change removes the existing time display (%H:%M) from status-right, but the PR title only mentions showing user/host. If the time is still desired, consider including it alongside user@host, or update the PR description/title to reflect the behavioral change.
home/modules/tmux/files/tmux.conf
Outdated
| status_left2="#S #{?client_prefix,#[fg=colour31]#[bg=colour234]#[nobold],#[fg=colour254]#[bg=colour234]#[nobold]}" | ||
| set -qg status-left $status_left1' '$status_left2 | ||
| set -qg status-right '%H:%M #[fg=colour16,bg=colour254]' | ||
| set -qg status-right ' #[fg=colour244]#(whoami)@#h #[fg=colour16,bg=colour254]' |
There was a problem hiding this comment.
The trailing style segment #[fg=colour16,bg=colour254] has no text following it, so it likely has no visible effect and makes the format harder to understand. Consider removing it or appending the intended content (e.g., if it was meant to style a time segment).
| set -qg status-right ' #[fg=colour244]#(whoami)@#h #[fg=colour16,bg=colour254]' | |
| set -qg status-right ' #[fg=colour244]#(whoami)@#h' |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| set -g @status_user "$USER" | ||
| set -qg status-right ' #[fg=colour244]#{@status_user}@#h' |
There was a problem hiding this comment.
This change replaces the previous clock (%H:%M) in status-right with user@host, which removes the time display entirely. If the intent is to add user/host rather than replace the existing clock, consider appending/prepending user@host while keeping %H:%M so the status bar doesn’t regress in functionality.
No description provided.