-
Notifications
You must be signed in to change notification settings - Fork 148
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
Conversation
`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
There was a problem hiding this 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 :)
tmuxcolors-256.conf
Outdated
@@ -3,25 +3,24 @@ | |||
# default statusbar colors | |||
set-option -g status-bg colour235 #base02 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set-option -g status-bg colour235 #base02 | |
set-option -g status-style bg=colour235 #base02 |
tmuxcolors-256.conf
Outdated
@@ -3,25 +3,24 @@ | |||
# default statusbar colors | |||
set-option -g status-bg colour235 #base02 | |||
set-option -g status-fg colour136 #yellow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set-option -g status-fg colour136 #yellow | |
set-option -g status-style fg=colour136 #yellow |
@antoineco - thanks for suggestion. I made those quick changes in |
…ven though the previous status-bg and status-fg are obsolete yet.
3870d07
to
fdb633f
Compare
I'm noticing a difference before and after, though I'm sure what's causing it. This PR fixes the And here's after: So previously my active window was red and the rest were gray, 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 |
tmuxcolors-256.conf
Outdated
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 |
There was a problem hiding this comment.
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:
set-option -g status-style bg=colour235 #base02 | |
set-option -g status-style bg=colour235,fg=colour136 #base02,yellow |
…nd. See comment in seebi#23
@oconnor663 - thanks for pointing out that problem. And @antoineco - thanks for hint on how to fix it. See latest commit 7849dba. |
Seems to fix the problem! Thanks for the incredibly rapid response :) |
ping @seebi , This pr looks great for now, let's get this merged? |
Since |
@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. |
Works great for me! Time to merge it? =) |
works for me as well |
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
andmessage-fg
have been removed from tmux.See tmux/tmux@f34ebfe