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

Update *.conf files to use applicable -style properties #23

Merged
merged 3 commits into from
Jul 14, 2019

Conversation

darcyparker
Copy link
Contributor

See #22

status-attr, window-status-fg, window-status-bg, window-status-current-fg, window-status-current-bg, pane-border-fg, pane-active-border-fg, message-bg and message-fg have been removed from tmux.

See tmux/tmux@f34ebfe

`status-attr`, `window-status-fg`, `window-status-bg`, `window-status-current-fg`, `window-status-current-bg`,
`pane-border-fg`, `pane-active-border-fg`, `message-bg` and `message-fg`
have been removed from tmux.
See tmux/tmux@f34ebfe
darcyparker added a commit to darcyparker/dotfiles that referenced this pull request Apr 23, 2019
tlvince added a commit to tlvince/tmux-config that referenced this pull request Apr 28, 2019
Copy link

@antoineco antoineco left a comment

Choose a reason for hiding this comment

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

I believe those could be updated as well for the sake of consistency. Tmux 2.9 doesn't complain about them, but it's just a matter of time :)

@@ -3,25 +3,24 @@
# default statusbar colors
set-option -g status-bg colour235 #base02

Choose a reason for hiding this comment

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

Suggested change
set-option -g status-bg colour235 #base02
set-option -g status-style bg=colour235 #base02

@@ -3,25 +3,24 @@
# default statusbar colors
set-option -g status-bg colour235 #base02
set-option -g status-fg colour136 #yellow

Choose a reason for hiding this comment

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

Suggested change
set-option -g status-fg colour136 #yellow
set-option -g status-style fg=colour136 #yellow

@darcyparker
Copy link
Contributor Author

darcyparker commented Apr 29, 2019

@antoineco - thanks for suggestion. I made those quick changes in
fdb633f

…ven though the previous status-bg and status-fg are obsolete yet.
darcyparker added a commit to darcyparker/dotfiles that referenced this pull request Apr 29, 2019
darcyparker added a commit to darcyparker/dotfiles that referenced this pull request Apr 29, 2019
@oconnor663
Copy link
Contributor

I'm noticing a difference before and after, though I'm sure what's causing it. This PR fixes the invalid option errors I was getting with the update to tmux 2.9, but the color change seems to happen regardless of whether I'm seeing errors. (My only color configuration is source-file tmuxcolors-dark.conf.) Here's the before:

image

And here's after:

image

So previously my active window was red and the rest were gray, but now everything's yellow. Is that expected?

@antoineco
Copy link

antoineco commented Apr 30, 2019

but now everything's yellow. Is that expected?

Nice catch, it's indeed not intended to be that way.

@darcyparker it works it you set all styles in 1 single command:

set-window-option -g window-status-current-style fg=brightred,bg=default

set-option -g status-bg colour235 #base02
set-option -g status-fg colour136 #yellow
set-option -g status-attr default
set-option -g status-style bg=colour235 #base02

Choose a reason for hiding this comment

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

The second command overrides the first, parameters must be inlined:

Suggested change
set-option -g status-style bg=colour235 #base02
set-option -g status-style bg=colour235,fg=colour136 #base02,yellow

@darcyparker
Copy link
Contributor Author

@oconnor663 - thanks for pointing out that problem. And @antoineco - thanks for hint on how to fix it. See latest commit 7849dba.

@oconnor663
Copy link
Contributor

Seems to fix the problem! Thanks for the incredibly rapid response :)

darcyparker added a commit to darcyparker/dotfiles that referenced this pull request Apr 30, 2019
@Xuanwo
Copy link

Xuanwo commented May 10, 2019

ping @seebi , This pr looks great for now, let's get this merged?

@rbjorklin
Copy link
Contributor

Since brightred is used to indicate the active window in window-status-current-style would it make sense to switch the colors around for pane numbers to be consistent? See these lines

@darcyparker
Copy link
Contributor Author

@rbjorklin - This pull request is about fixing: "Invalid options that were deprecated and removed from tmux" #22

I made no change to the lines you referred to and there was not problem with those options being deprecated/removed. Compare it to what's on master: https://github.com/seebi/tmux-colors-solarized/blob/master/tmuxcolors-dark.conf#L26-L28

I am not sure about why the existing/current color choices are the way they are... It could be interesting to explore the idea of switching the colors as you suggested. But I would make it a separate pull request/issue. The scope of that change is different than what this pull request is solving.

@marczych
Copy link

Works great for me! Time to merge it? =)

@seebi
Copy link
Owner

seebi commented Jul 14, 2019

works for me as well

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.

8 participants