Fix titles and detect edges for hide_edge_borders

3 files changed, 96 insertions, 102 deletions

diff --git a/include/sway/tree/view.h b/include/sway/tree/view.h
index 8091fe0c..de431c89 100644
--- a/include/sway/tree/view.h
+++ b/include/sway/tree/view.h
@@ -51,6 +51,10 @@ struct sway_view {
 	char *title_format;
 	enum sway_container_border border;
 	int border_thickness;
+	bool border_top;
+	bool border_bottom;
+	bool border_left;
+	bool border_right;
 
 	union {
 		struct wlr_xdg_surface_v6 *wlr_xdg_surface_v6;
diff --git a/sway/desktop/output.c b/sway/desktop/output.c
index 974cd56c..705e542a 100644
--- a/sway/desktop/output.c
+++ b/sway/desktop/output.c
@@ -322,15 +322,7 @@ static void render_container_simple_border_normal(struct sway_output *output,
 	struct wlr_box box;
 	float color[4];
 
-	int other_views = 1;
-	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)) {
+	if (con->sway_view->border_left) {
 		// Child border - left edge
 		memcpy(&color, colors->child_border, sizeof(float) * 4);
 		color[3] *= con->alpha;
@@ -339,7 +331,9 @@ static void render_container_simple_border_normal(struct sway_output *output,
 		box.width = con->sway_view->border_thickness;
 		box.height = con->height - 1;
 		render_rect(output->wlr_output, output_damage, &box, color);
+	}
 
+	if (con->sway_view->border_right) {
 		// Child border - right edge
 		if (con->parent->children->length == 1
 				&& con->parent->layout == L_HORIZ) {
@@ -355,9 +349,7 @@ static void render_container_simple_border_normal(struct sway_output *output,
 		render_rect(output->wlr_output, output_damage, &box, color);
 	}
 
-	if (config->hide_edge_borders != E_HORIZONTAL
-			&& config->hide_edge_borders != E_BOTH
-			&& (config->hide_edge_borders != E_SMART || other_views)) {
+	if (con->sway_view->border_bottom) {
 		// Child border - bottom edge
 		if (con->parent->children->length == 1
 				&& con->parent->layout == L_VERT) {
@@ -371,7 +363,9 @@ static void render_container_simple_border_normal(struct sway_output *output,
 		box.width = con->width;
 		box.height = con->sway_view->border_thickness;
 		render_rect(output->wlr_output, output_damage, &box, color);
+	}
 
+	if (con->sway_view->border_top) {
 		// Single pixel bar above title
 		memcpy(&color, colors->border, sizeof(float) * 4);
 		color[3] *= con->alpha;
@@ -380,41 +374,46 @@ static void render_container_simple_border_normal(struct sway_output *output,
 		box.width = con->width;
 		box.height = 1;
 		render_rect(output->wlr_output, output_damage, &box, color);
+	}
 
-		// Single pixel bar below title
-		box.x = con->x + con->sway_view->border_thickness;
-		box.y = con->sway_view->y - 1;
-		box.width = con->width - con->sway_view->border_thickness * 2;
-		box.height = 1;
-		render_rect(output->wlr_output, output_damage, &box, color);
-
-		// Title background
-		memcpy(&color, colors->background, sizeof(float) * 4);
-		color[3] *= con->alpha;
-		box.x = con->x + con->sway_view->border_thickness;
-		box.y = con->y + 1;
-		box.width = con->width - con->sway_view->border_thickness * 2;
-		box.height = con->sway_view->y - con->y - 2;
-		render_rect(output->wlr_output, output_damage, &box, color);
-
-		// Title text
-		if (title_texture) {
-			float output_scale = output->wlr_output->scale;
-			struct wlr_box texture_box = {
-				.x = box.x * output_scale,
-				.y = box.y * output_scale,
-			};
-			wlr_texture_get_size(title_texture,
-				&texture_box.width, &texture_box.height);
-
-			float matrix[9];
-			wlr_matrix_project_box(matrix, &texture_box,
-				WL_OUTPUT_TRANSFORM_NORMAL,
-				0.0, output->wlr_output->transform_matrix);
-
-			render_texture(output->wlr_output, output_damage, title_texture,
-				&texture_box, matrix, 1.0);
-		}
+	// 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.y = con->sway_view->y - 1;
+	box.width = con->width - con->sway_view->border_thickness * 2;
+	box.height = 1;
+	render_rect(output->wlr_output, output_damage, &box, color);
+
+	// Title background
+	memcpy(&color, colors->background, sizeof(float) * 4);
+	color[3] *= con->alpha;
+	box.x = con->x
+		+ con->sway_view->border_thickness * con->sway_view->border_left;
+	box.y = con->y + con->sway_view->border_top;
+	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 - (con->sway_view->border_top + 1);
+	render_rect(output->wlr_output, output_damage, &box, color);
+
+	// Title text
+	if (title_texture) {
+		float output_scale = output->wlr_output->scale;
+		struct wlr_box texture_box = {
+			.x = box.x * output_scale,
+			.y = box.y * output_scale,
+		};
+		wlr_texture_get_size(title_texture,
+			&texture_box.width, &texture_box.height);
+
+		float matrix[9];
+		wlr_matrix_project_box(matrix, &texture_box,
+			WL_OUTPUT_TRANSFORM_NORMAL,
+			0.0, output->wlr_output->transform_matrix);
+
+		render_texture(output->wlr_output, output_damage, title_texture,
+			&texture_box, matrix, 1.0);
 	}
 }
 
@@ -427,15 +426,7 @@ static void render_container_simple_border_pixel(struct sway_output *output,
 	struct wlr_box box;
 	float color[4];
 
-	int other_views = 1;
-	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)) {
+	if (con->sway_view->border_left) {
 		// Child border - left edge
 		memcpy(&color, colors->child_border, sizeof(float) * 4);
 		color[3] *= con->alpha;
@@ -444,7 +435,9 @@ static void render_container_simple_border_pixel(struct sway_output *output,
 		box.width = con->sway_view->border_thickness;
 		box.height = con->height;
 		render_rect(output->wlr_output, output_damage, &box, color);
+	}
 
+	if (con->sway_view->border_right) {
 		// Child border - right edge
 		if (con->parent->children->length == 1
 				&& con->parent->layout == L_HORIZ) {
@@ -460,16 +453,18 @@ static void render_container_simple_border_pixel(struct sway_output *output,
 		render_rect(output->wlr_output, output_damage, &box, color);
 	}
 
-	if (config->hide_edge_borders != E_HORIZONTAL
-			&& config->hide_edge_borders != E_BOTH
-			&& (config->hide_edge_borders != E_SMART || other_views)) {
+	if (con->sway_view->border_top) {
 		// Child border - top edge
+		memcpy(&color, colors->child_border, sizeof(float) * 4);
+		color[3] *= con->alpha;
 		box.x = con->x;
 		box.y = con->y;
 		box.width = con->width;
 		box.height = con->sway_view->border_thickness;
 		render_rect(output->wlr_output, output_damage, &box, color);
+	}
 
+	if (con->sway_view->border_bottom) {
 		// Child border - bottom edge
 		if (con->parent->children->length == 1
 				&& con->parent->layout == L_VERT) {
diff --git a/sway/tree/view.c b/sway/tree/view.c
index e2cb8a7a..a48a6619 100644
--- a/sway/tree/view.c
+++ b/sway/tree/view.c
@@ -93,8 +93,9 @@ void view_autoconfigure(struct sway_view *view) {
 		return;
 	}
 
+	struct sway_container *output = container_parent(view->swayc, C_OUTPUT);
+
 	if (view->is_fullscreen) {
-		struct sway_container *output = container_parent(view->swayc, C_OUTPUT);
 		view_configure(view, 0, 0, output->width, output->height);
 		view->x = view->y = 0;
 		return;
@@ -106,6 +107,25 @@ void view_autoconfigure(struct sway_view *view) {
 		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;
 	x = y = width = height = 0;
 	switch (view->border) {
@@ -116,51 +136,26 @@ void view_autoconfigure(struct sway_view *view) {
 		height = view->swayc->height;
 		break;
 	case B_PIXEL:
-		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)) {
-			y = view->swayc->y + view->border_thickness;
-			height = view->swayc->height - view->border_thickness * 2;
-		} else {
-			y = view->swayc->y;
-			height = view->swayc->height;
-		}
+		x = view->swayc->x + view->border_thickness * view->border_left;
+		y = view->swayc->y + view->border_thickness * view->border_top;
+		width = view->swayc->width
+			- view->border_thickness * view->border_left
+			- view->border_thickness * view->border_right;
+		height = view->swayc->height
+			- view->border_thickness * view->border_top
+			- view->border_thickness * view->border_bottom;
 		break;
 	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
-			y = view->swayc->y + config->font_height
-				+ view->border_thickness * 2;
-			height = view->swayc->height - config->font_height
-				- view->border_thickness * 3;
-		} else {
-			y = view->swayc->y;
-			height = view->swayc->height;
-		}
+		// Height is: border + title height + border + view height + border
+		x = view->swayc->x + view->border_thickness * view->border_left;
+		y = view->swayc->y + config->font_height
+			+ view->border_thickness * (view->border_top + 1);
+		width = view->swayc->width
+			- view->border_thickness * view->border_left
+			- view->border_thickness * view->border_right;
+		height = view->swayc->height - config->font_height
+			- view->border_thickness * (view->border_top + 1)
+			- view->border_thickness * view->border_bottom;
 		break;
 	}