Update titlebar borders to match i3's current behavior.

This commit is contained in:
Dark 2019-08-23 22:44:53 -04:00 committed by Brian Ashworth
parent 90bef0364b
commit c3fbb01e07

View file

@ -343,8 +343,7 @@ static void render_view(struct sway_output *output, pixman_region32_t *damage,
* otherwise the colors will be incorrect when using opacity. * otherwise the colors will be incorrect when using opacity.
* *
* The height is: 1px border, 3px padding, font height, 3px padding, 1px border * The height is: 1px border, 3px padding, font height, 3px padding, 1px border
* The left side for L_TABBED is: 1px border, 2px padding, title * The left side is: 1px border, 2px padding, title
* The left side for other layouts is: 3px padding, title
*/ */
static void render_titlebar(struct sway_output *output, static void render_titlebar(struct sway_output *output,
pixman_region32_t *output_damage, struct sway_container *con, pixman_region32_t *output_damage, struct sway_container *con,
@ -353,12 +352,7 @@ static void render_titlebar(struct sway_output *output,
struct wlr_texture *marks_texture) { struct wlr_texture *marks_texture) {
struct wlr_box box; struct wlr_box box;
float color[4]; float color[4];
struct sway_container_state *state = &con->current;
float output_scale = output->wlr_output->scale; float output_scale = output->wlr_output->scale;
enum sway_container_layout layout = container_current_parent_layout(con);
list_t *children = container_get_current_siblings(con);
bool is_last_child = children->length == 0 ||
children->items[children->length - 1] == con;
double output_x = output->lx; double output_x = output->lx;
double output_y = output->ly; double output_y = output->ly;
int titlebar_border_thickness = config->titlebar_border_thickness; int titlebar_border_thickness = config->titlebar_border_thickness;
@ -377,42 +371,28 @@ static void render_titlebar(struct sway_output *output,
render_rect(output, output_damage, &box, color); render_rect(output, output_damage, &box, color);
// Single pixel bar below title // Single pixel bar below title
size_t left_offset = 0, right_offset = 0; box.x = x;
bool connects_sides = false;
if (layout == L_HORIZ || layout == L_VERT ||
(layout == L_STACKED && is_last_child)) {
if (con->view) {
left_offset = state->border_left * state->border_thickness;
right_offset = state->border_right * state->border_thickness;
connects_sides = true;
}
}
box.x = x + left_offset;
box.y = y + container_titlebar_height() - titlebar_border_thickness; box.y = y + container_titlebar_height() - titlebar_border_thickness;
box.width = width - left_offset - right_offset; box.width = width;
box.height = titlebar_border_thickness; box.height = titlebar_border_thickness;
scale_box(&box, output_scale); scale_box(&box, output_scale);
render_rect(output, output_damage, &box, color); render_rect(output, output_damage, &box, color);
if (layout == L_TABBED) { // Single pixel left edge
// Single pixel left edge box.x = x;
box.x = x; box.y = y + titlebar_border_thickness;
box.y = y + titlebar_border_thickness; box.width = titlebar_border_thickness;
box.width = titlebar_border_thickness; box.height = container_titlebar_height() - titlebar_border_thickness * 2;
box.height = scale_box(&box, output_scale);
container_titlebar_height() - titlebar_border_thickness * 2; render_rect(output, output_damage, &box, color);
scale_box(&box, output_scale);
render_rect(output, output_damage, &box, color);
// Single pixel right edge // Single pixel right edge
box.x = x + width - titlebar_border_thickness; box.x = x + width - titlebar_border_thickness;
box.y = y + titlebar_border_thickness; box.y = y + titlebar_border_thickness;
box.width = titlebar_border_thickness; box.width = titlebar_border_thickness;
box.height = box.height = container_titlebar_height() - titlebar_border_thickness * 2;
container_titlebar_height() - titlebar_border_thickness * 2; scale_box(&box, output_scale);
scale_box(&box, output_scale); render_rect(output, output_damage, &box, color);
render_rect(output, output_damage, &box, color);
}
int inner_x = x - output_x + titlebar_h_padding; int inner_x = x - output_x + titlebar_h_padding;
int bg_y = y + titlebar_border_thickness; int bg_y = y + titlebar_border_thickness;
@ -581,10 +561,9 @@ static void render_titlebar(struct sway_output *output,
} }
// Padding on left side // Padding on left side
left_offset = (layout == L_TABBED) * titlebar_border_thickness; box.x = x + titlebar_border_thickness;
box.x = x + left_offset;
box.y = y + titlebar_border_thickness; box.y = y + titlebar_border_thickness;
box.width = titlebar_h_padding - left_offset; box.width = titlebar_h_padding - titlebar_border_thickness;
box.height = (titlebar_v_padding - titlebar_border_thickness) * 2 + box.height = (titlebar_v_padding - titlebar_border_thickness) * 2 +
config->font_height; config->font_height;
scale_box(&box, output_scale); scale_box(&box, output_scale);
@ -595,10 +574,9 @@ static void render_titlebar(struct sway_output *output,
render_rect(output, output_damage, &box, color); render_rect(output, output_damage, &box, color);
// Padding on right side // Padding on right side
right_offset = (layout == L_TABBED) * titlebar_border_thickness;
box.x = x + width - titlebar_h_padding; box.x = x + width - titlebar_h_padding;
box.y = y + titlebar_border_thickness; box.y = y + titlebar_border_thickness;
box.width = titlebar_h_padding - right_offset; box.width = titlebar_h_padding - titlebar_border_thickness;
box.height = (titlebar_v_padding - titlebar_border_thickness) * 2 + box.height = (titlebar_v_padding - titlebar_border_thickness) * 2 +
config->font_height; config->font_height;
scale_box(&box, output_scale); scale_box(&box, output_scale);
@ -608,24 +586,6 @@ static void render_titlebar(struct sway_output *output,
box.x = right_rx; box.x = right_rx;
} }
render_rect(output, output_damage, &box, color); render_rect(output, output_damage, &box, color);
if (connects_sides) {
// Left pixel in line with bottom bar
box.x = x;
box.y = y + container_titlebar_height() - titlebar_border_thickness;
box.width = state->border_thickness * state->border_left;
box.height = titlebar_border_thickness;
scale_box(&box, output_scale);
render_rect(output, output_damage, &box, color);
// Right pixel in line with bottom bar
box.x = x + width - state->border_thickness * state->border_right;
box.y = y + container_titlebar_height() - titlebar_border_thickness;
box.width = state->border_thickness * state->border_right;
box.height = titlebar_border_thickness;
scale_box(&box, output_scale);
render_rect(output, output_damage, &box, color);
}
} }
/** /**