Skip to content

Polish tmux Macchiato status bar best practices#20

Closed
hdp617 wants to merge 2 commits into
mainfrom
cursor/tmux-macchiato-best-practices-3e49
Closed

Polish tmux Macchiato status bar best practices#20
hdp617 wants to merge 2 commits into
mainfrom
cursor/tmux-macchiato-best-practices-3e49

Conversation

@hdp617

@hdp617 hdp617 commented Jul 16, 2026

Copy link
Copy Markdown
Owner

Summary

PR #7 is already merged and there were no uncommitted local changes. This follow-up applies remaining best-practice polish to the landed Catppuccin Macchiato tmux config as two independent commits.

Commits

  1. Harden @machine_name caching — first line of ~/.name, fallback to hostname -s, strip CR/LF so a trailing newline does not break status-right
  2. Clarify status-right prefix conditional — explicit true/false fg in #{?client_prefix,…} instead of set-then-override

Test plan

  • tmux source-file ~/.tmux.conf / C-a C-r
  • Status bar shows hostname or ~/.name without an extra blank line gap
  • Holding prefix turns the date fg red (#ed8796); release returns peach (#f5a97f)
Open in Web Open in Cursor 

cursoragent and others added 2 commits July 16, 2026 05:03
Prefer the first line of ~/.name, fall back to hostname -s, and strip
CR/LF so a trailing newline from cat does not distort status-right.

Co-authored-by: Huy Pham <hdp617@users.noreply.github.com>
Use an explicit true/false fg in #{?client_prefix,…} instead of setting
peach then optionally overriding, so prefix-active styling is obvious.

Co-authored-by: Huy Pham <hdp617@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 16, 2026 05:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@hdp617 hdp617 closed this Jul 16, 2026
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