swaybar: obey height if given

If there is a bar height given, use that as the height rather than as a
minimum height. This matches i3-gaps behavior.
This commit is contained in:
Brian Ashworth 2019-01-09 20:58:00 -05:00
parent 14cab78612
commit 124085eba9
3 changed files with 15 additions and 9 deletions

View file

@ -96,7 +96,7 @@ struct bar_config *default_bar_config(void) {
bar->pango_markup = false; bar->pango_markup = false;
bar->swaybar_command = NULL; bar->swaybar_command = NULL;
bar->font = NULL; bar->font = NULL;
bar->height = -1; bar->height = 0;
bar->workspace_buttons = true; bar->workspace_buttons = true;
bar->wrap_scroll = false; bar->wrap_scroll = false;
bar->separator_symbol = NULL; bar->separator_symbol = NULL;

View file

@ -69,7 +69,7 @@ Sway allows configuring swaybar in the sway configuration file.
use. use.
*height* <height> *height* <height>
Sets the height of the bar. Default height will match the font size. Sets the height of the bar. Default height (0) will match the font size.
*bindsym* [--release] button<n> <command> *bindsym* [--release] button<n> <command>
Executes _command_ when mouse button _n_ has been pressed (or if _released_ Executes _command_ when mouse button _n_ has been pressed (or if _released_

View file

@ -44,7 +44,8 @@ static uint32_t render_status_line_error(cairo_t *cairo,
uint32_t ideal_height = text_height + ws_vertical_padding * 2; uint32_t ideal_height = text_height + ws_vertical_padding * 2;
uint32_t ideal_surface_height = ideal_height / output->scale; uint32_t ideal_surface_height = ideal_height / output->scale;
if (output->height < ideal_surface_height) { if (!output->bar->config->height &&
output->height < ideal_surface_height) {
return ideal_surface_height; return ideal_surface_height;
} }
*x -= text_width + margin; *x -= text_width + margin;
@ -76,7 +77,8 @@ static uint32_t render_status_line_text(cairo_t *cairo,
uint32_t ideal_height = text_height + ws_vertical_padding * 2; uint32_t ideal_height = text_height + ws_vertical_padding * 2;
uint32_t ideal_surface_height = ideal_height / output->scale; uint32_t ideal_surface_height = ideal_height / output->scale;
if (output->height < ideal_surface_height) { if (!output->bar->config->height &&
output->height < ideal_surface_height) {
return ideal_surface_height; return ideal_surface_height;
} }
@ -151,7 +153,8 @@ static uint32_t render_status_block(cairo_t *cairo,
double block_width = width; double block_width = width;
uint32_t ideal_height = text_height + ws_vertical_padding * 2; uint32_t ideal_height = text_height + ws_vertical_padding * 2;
uint32_t ideal_surface_height = ideal_height / output->scale; uint32_t ideal_surface_height = ideal_height / output->scale;
if (output->height < ideal_surface_height) { if (!output->bar->config->height &&
output->height < ideal_surface_height) {
return ideal_surface_height; return ideal_surface_height;
} }
@ -173,7 +176,8 @@ static uint32_t render_status_block(cairo_t *cairo,
output->scale, false, "%s", config->sep_symbol); output->scale, false, "%s", config->sep_symbol);
uint32_t _ideal_height = sep_height + ws_vertical_padding * 2; uint32_t _ideal_height = sep_height + ws_vertical_padding * 2;
uint32_t _ideal_surface_height = _ideal_height / output->scale; uint32_t _ideal_surface_height = _ideal_height / output->scale;
if (output->height < _ideal_surface_height) { if (!output->bar->config->height &&
output->height < _ideal_surface_height) {
return _ideal_surface_height; return _ideal_surface_height;
} }
if (sep_width > sep_block_width) { if (sep_width > sep_block_width) {
@ -328,7 +332,8 @@ static uint32_t render_binding_mode_indicator(cairo_t *cairo,
uint32_t ideal_height = text_height + ws_vertical_padding * 2 uint32_t ideal_height = text_height + ws_vertical_padding * 2
+ border_width * 2; + border_width * 2;
uint32_t ideal_surface_height = ideal_height / output->scale; uint32_t ideal_surface_height = ideal_height / output->scale;
if (output->height < ideal_surface_height) { if (!output->bar->config->height &&
output->height < ideal_surface_height) {
return ideal_surface_height; return ideal_surface_height;
} }
uint32_t width = text_width + ws_horizontal_padding * 2 + border_width * 2; uint32_t width = text_width + ws_horizontal_padding * 2 + border_width * 2;
@ -394,7 +399,8 @@ static uint32_t render_workspace_button(cairo_t *cairo,
uint32_t ideal_height = ws_vertical_padding * 2 + text_height uint32_t ideal_height = ws_vertical_padding * 2 + text_height
+ border_width * 2; + border_width * 2;
uint32_t ideal_surface_height = ideal_height / output->scale; uint32_t ideal_surface_height = ideal_height / output->scale;
if (output->height < ideal_surface_height) { if (!output->bar->config->height &&
output->height < ideal_surface_height) {
return ideal_surface_height; return ideal_surface_height;
} }
@ -521,7 +527,7 @@ void render_frame(struct swaybar_output *output) {
cairo_restore(cairo); cairo_restore(cairo);
uint32_t height = render_to_cairo(cairo, output); uint32_t height = render_to_cairo(cairo, output);
int config_height = output->bar->config->height; int config_height = output->bar->config->height;
if (config_height >= 0 && height < (uint32_t)config_height) { if (config_height > 0) {
height = config_height; height = config_height;
} }
if (height != output->height || output->width == 0) { if (height != output->height || output->width == 0) {