Merge pull request #1978 from RedSoxFan/fix-1975

Fix #1975
This commit is contained in:
Drew DeVault 2018-05-14 18:10:54 -04:00 committed by GitHub
commit f1f54bbc88
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 96 additions and 106 deletions

View file

@ -55,6 +55,10 @@ struct sway_view {
char *title_format; char *title_format;
enum sway_container_border border; enum sway_container_border border;
int border_thickness; int border_thickness;
bool border_top;
bool border_bottom;
bool border_left;
bool border_right;
list_t *executed_criteria; list_t *executed_criteria;

View file

@ -321,15 +321,7 @@ static void render_container_simple_border_normal(struct sway_output *output,
struct wlr_box box; struct wlr_box box;
float color[4]; float color[4];
int other_views = 1; if (con->sway_view->border_left) {
if (config->hide_edge_borders == E_SMART) {
struct sway_container *ws = container_parent(con, C_WORKSPACE);
other_views = container_count_descendants_of_type(ws, C_VIEW) - 1;
}
if (config->hide_edge_borders != E_VERTICAL
&& config->hide_edge_borders != E_BOTH
&& (config->hide_edge_borders != E_SMART || other_views)) {
// Child border - left edge // Child border - left edge
memcpy(&color, colors->child_border, sizeof(float) * 4); memcpy(&color, colors->child_border, sizeof(float) * 4);
color[3] *= con->alpha; color[3] *= con->alpha;
@ -338,7 +330,9 @@ static void render_container_simple_border_normal(struct sway_output *output,
box.width = con->sway_view->border_thickness; box.width = con->sway_view->border_thickness;
box.height = con->height - 1; box.height = con->height - 1;
render_rect(output->wlr_output, output_damage, &box, color); render_rect(output->wlr_output, output_damage, &box, color);
}
if (con->sway_view->border_right) {
// Child border - right edge // Child border - right edge
if (con->parent->children->length == 1 if (con->parent->children->length == 1
&& con->parent->layout == L_HORIZ) { && con->parent->layout == L_HORIZ) {
@ -354,9 +348,7 @@ static void render_container_simple_border_normal(struct sway_output *output,
render_rect(output->wlr_output, output_damage, &box, color); render_rect(output->wlr_output, output_damage, &box, color);
} }
if (config->hide_edge_borders != E_HORIZONTAL if (con->sway_view->border_bottom) {
&& config->hide_edge_borders != E_BOTH
&& (config->hide_edge_borders != E_SMART || other_views)) {
// Child border - bottom edge // Child border - bottom edge
if (con->parent->children->length == 1 if (con->parent->children->length == 1
&& con->parent->layout == L_VERT) { && con->parent->layout == L_VERT) {
@ -370,6 +362,7 @@ static void render_container_simple_border_normal(struct sway_output *output,
box.width = con->width; box.width = con->width;
box.height = con->sway_view->border_thickness; box.height = con->sway_view->border_thickness;
render_rect(output->wlr_output, output_damage, &box, color); render_rect(output->wlr_output, output_damage, &box, color);
}
// Single pixel bar above title // Single pixel bar above title
memcpy(&color, colors->border, sizeof(float) * 4); memcpy(&color, colors->border, sizeof(float) * 4);
@ -381,6 +374,8 @@ static void render_container_simple_border_normal(struct sway_output *output,
render_rect(output->wlr_output, output_damage, &box, color); render_rect(output->wlr_output, output_damage, &box, color);
// Single pixel bar below title // Single pixel bar below title
memcpy(&color, colors->border, sizeof(float) * 4);
color[3] *= con->alpha;
box.x = con->x + con->sway_view->border_thickness; box.x = con->x + con->sway_view->border_thickness;
box.y = con->sway_view->y - 1; box.y = con->sway_view->y - 1;
box.width = con->width - con->sway_view->border_thickness * 2; box.width = con->width - con->sway_view->border_thickness * 2;
@ -390,9 +385,12 @@ static void render_container_simple_border_normal(struct sway_output *output,
// Title background // Title background
memcpy(&color, colors->background, sizeof(float) * 4); memcpy(&color, colors->background, sizeof(float) * 4);
color[3] *= con->alpha; color[3] *= con->alpha;
box.x = con->x + con->sway_view->border_thickness; box.x = con->x
+ con->sway_view->border_thickness * con->sway_view->border_left;
box.y = con->y + 1; box.y = con->y + 1;
box.width = con->width - con->sway_view->border_thickness * 2; box.width = con->width
- con->sway_view->border_thickness * con->sway_view->border_left
- con->sway_view->border_thickness * con->sway_view->border_right;
box.height = con->sway_view->y - con->y - 2; box.height = con->sway_view->y - con->y - 2;
render_rect(output->wlr_output, output_damage, &box, color); render_rect(output->wlr_output, output_damage, &box, color);
@ -415,7 +413,6 @@ static void render_container_simple_border_normal(struct sway_output *output,
&texture_box, matrix, 1.0); &texture_box, matrix, 1.0);
} }
} }
}
/** /**
* Render decorations for a view with "border pixel". * Render decorations for a view with "border pixel".
@ -426,15 +423,7 @@ static void render_container_simple_border_pixel(struct sway_output *output,
struct wlr_box box; struct wlr_box box;
float color[4]; float color[4];
int other_views = 1; if (con->sway_view->border_left) {
if (config->hide_edge_borders == E_SMART) {
struct sway_container *ws = container_parent(con, C_WORKSPACE);
other_views = container_count_descendants_of_type(ws, C_VIEW) - 1;
}
if (config->hide_edge_borders != E_VERTICAL
&& config->hide_edge_borders != E_BOTH
&& (config->hide_edge_borders != E_SMART || other_views)) {
// Child border - left edge // Child border - left edge
memcpy(&color, colors->child_border, sizeof(float) * 4); memcpy(&color, colors->child_border, sizeof(float) * 4);
color[3] *= con->alpha; color[3] *= con->alpha;
@ -443,7 +432,9 @@ static void render_container_simple_border_pixel(struct sway_output *output,
box.width = con->sway_view->border_thickness; box.width = con->sway_view->border_thickness;
box.height = con->height; box.height = con->height;
render_rect(output->wlr_output, output_damage, &box, color); render_rect(output->wlr_output, output_damage, &box, color);
}
if (con->sway_view->border_right) {
// Child border - right edge // Child border - right edge
if (con->parent->children->length == 1 if (con->parent->children->length == 1
&& con->parent->layout == L_HORIZ) { && con->parent->layout == L_HORIZ) {
@ -459,16 +450,18 @@ static void render_container_simple_border_pixel(struct sway_output *output,
render_rect(output->wlr_output, output_damage, &box, color); render_rect(output->wlr_output, output_damage, &box, color);
} }
if (config->hide_edge_borders != E_HORIZONTAL if (con->sway_view->border_top) {
&& config->hide_edge_borders != E_BOTH
&& (config->hide_edge_borders != E_SMART || other_views)) {
// Child border - top edge // Child border - top edge
memcpy(&color, colors->child_border, sizeof(float) * 4);
color[3] *= con->alpha;
box.x = con->x; box.x = con->x;
box.y = con->y; box.y = con->y;
box.width = con->width; box.width = con->width;
box.height = con->sway_view->border_thickness; box.height = con->sway_view->border_thickness;
render_rect(output->wlr_output, output_damage, &box, color); render_rect(output->wlr_output, output_damage, &box, color);
}
if (con->sway_view->border_bottom) {
// Child border - bottom edge // Child border - bottom edge
if (con->parent->children->length == 1 if (con->parent->children->length == 1
&& con->parent->layout == L_VERT) { && con->parent->layout == L_VERT) {

View file

@ -120,8 +120,9 @@ void view_autoconfigure(struct sway_view *view) {
return; return;
} }
if (view->is_fullscreen) {
struct sway_container *output = container_parent(view->swayc, C_OUTPUT); struct sway_container *output = container_parent(view->swayc, C_OUTPUT);
if (view->is_fullscreen) {
view_configure(view, 0, 0, output->width, output->height); view_configure(view, 0, 0, output->width, output->height);
view->x = view->y = 0; view->x = view->y = 0;
return; return;
@ -133,6 +134,25 @@ void view_autoconfigure(struct sway_view *view) {
other_views = container_count_descendants_of_type(ws, C_VIEW) - 1; other_views = container_count_descendants_of_type(ws, C_VIEW) - 1;
} }
view->border_top = view->border_bottom = true;
view->border_left = view->border_right = true;
if (view->swayc->layout != L_FLOATING) {
if (config->hide_edge_borders == E_BOTH
|| config->hide_edge_borders == E_VERTICAL
|| (config->hide_edge_borders == E_SMART && !other_views)) {
view->border_left = view->swayc->x != 0;
int right_x = view->swayc->x + view->swayc->width;
view->border_right = right_x != output->width;
}
if (config->hide_edge_borders == E_BOTH
|| config->hide_edge_borders == E_HORIZONTAL
|| (config->hide_edge_borders == E_SMART && !other_views)) {
view->border_top = view->swayc->y != 0;
int bottom_y = view->swayc->y + view->swayc->height;
view->border_bottom = bottom_y != output->height;
}
}
double x, y, width, height; double x, y, width, height;
x = y = width = height = 0; x = y = width = height = 0;
switch (view->border) { switch (view->border) {
@ -143,51 +163,24 @@ void view_autoconfigure(struct sway_view *view) {
height = view->swayc->height; height = view->swayc->height;
break; break;
case B_PIXEL: case B_PIXEL:
if (view->swayc->layout > L_VERT x = view->swayc->x + view->border_thickness * view->border_left;
|| config->hide_edge_borders == E_NONE y = view->swayc->y + view->border_thickness * view->border_top;
|| config->hide_edge_borders == E_HORIZONTAL width = view->swayc->width
|| (config->hide_edge_borders == E_SMART && other_views)) { - view->border_thickness * view->border_left
x = view->swayc->x + view->border_thickness; - view->border_thickness * view->border_right;
width = view->swayc->width - view->border_thickness * 2; height = view->swayc->height
} else { - view->border_thickness * view->border_top
x = view->swayc->x; - view->border_thickness * view->border_bottom;
width = view->swayc->width;
}
if (view->swayc->layout > L_VERT
|| config->hide_edge_borders == E_NONE
|| config->hide_edge_borders == E_VERTICAL
|| (config->hide_edge_borders == E_SMART && other_views)) {
y = view->swayc->y + view->border_thickness;
height = view->swayc->height - view->border_thickness * 2;
} else {
y = view->swayc->y;
height = view->swayc->height;
}
break; break;
case B_NORMAL: case B_NORMAL:
if (view->swayc->layout > L_VERT
|| config->hide_edge_borders == E_NONE
|| config->hide_edge_borders == E_HORIZONTAL
|| (config->hide_edge_borders == E_SMART && other_views)) {
x = view->swayc->x + view->border_thickness;
width = view->swayc->width - view->border_thickness * 2;
} else {
x = view->swayc->x;
width = view->swayc->width;
}
if (view->swayc->layout > L_VERT
|| config->hide_edge_borders == E_NONE
|| config->hide_edge_borders == E_VERTICAL
|| (config->hide_edge_borders == E_SMART && other_views)) {
// Height is: border + title height + border + view height + border // Height is: border + title height + border + view height + border
y = view->swayc->y + config->font_height x = view->swayc->x + view->border_thickness * view->border_left;
+ view->border_thickness * 2; y = view->swayc->y + config->font_height + view->border_thickness * 2;
width = view->swayc->width
- view->border_thickness * view->border_left
- view->border_thickness * view->border_right;
height = view->swayc->height - config->font_height height = view->swayc->height - config->font_height
- view->border_thickness * 3; - view->border_thickness * (2 + view->border_bottom);
} else {
y = view->swayc->y;
height = view->swayc->height;
}
break; break;
} }