Skip to content

fix(status): take into account reverse style - #102

Open
kruton wants to merge 1 commit into
nordtheme:developfrom
kruton:reverse-color-fix
Open

fix(status): take into account reverse style#102
kruton wants to merge 1 commit into
nordtheme:developfrom
kruton:reverse-color-fix

Conversation

@kruton

@kruton kruton commented Jun 23, 2026

Copy link
Copy Markdown

When the tmux activity style is set as "reverse" (the default) then activity in any window renders that section of the status bar as the wrong color. Take into account the rendering style when drawing those window separators.

Fixes #101

When the tmux activity style is set as "reverse" (the default) then
activity in any window renders that section of the status bar as
the wrong color. Take into account the rendering style when drawing
those window separators.
@kruton
kruton requested a review from svengreb as a code owner June 23, 2026 18:06
Copilot AI review requested due to automatic review settings June 23, 2026 18:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adjusts Nord tmux’s window status formatting so Powerline-style separators render with correct colors when window alert styles use reverse (the tmux default for activity/bell/silence), fixing the swapped/misaligned separator colors reported in #101.

Changes:

  • Make window-status-format use computed colors (via tmux environment variables) instead of fixed black/brightblack so separators/background adapt during alerts.
  • Update both patched-font and no-patched-font status content variants to use the new computed window background.
  • Compute and export NORD_TMUX_WINDOW_BG and NORD_TMUX_TRANSITION_COLOR in nord.tmux based on alert flags and reverse-style detection.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/nord-status-content.conf Uses computed transition/window background colors in the Powerline window status format to avoid swapped separator colors under reverse.
src/nord-status-content-no-patched-font.conf Aligns window background color with computed alert-aware background even without Powerline glyphs.
nord.tmux Introduces env vars used by the status content to dynamically pick correct colors for alert windows when reverse is in effect.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread nord.tmux Outdated
@kruton
kruton force-pushed the reverse-color-fix branch from 98740a8 to a91f437 Compare June 23, 2026 21:12
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.

Broken status bar separators on window alerts (activity, bell, silence)

2 participants