Use constants for titlebar dimensions

3 files changed, 15 insertions, 11 deletions

diff --git a/sway/tree/arrange.c b/sway/tree/arrange.c
index b8e07bca..37f4a066 100644
--- a/sway/tree/arrange.c
+++ b/sway/tree/arrange.c
@@ -88,10 +88,10 @@ static void apply_horiz_layout(struct sway_container *parent) {
 	}
 	size_t parent_offset = 0;
 	if (parent->parent->layout == L_TABBED) {
-		parent_offset = config->font_height + 8;
+		parent_offset = container_titlebar_height();
 	} else if (parent->parent->layout == L_STACKED) {
-		parent_offset = (config->font_height + 8)
-			* parent->parent->children->length;
+		parent_offset =
+			container_titlebar_height() * parent->parent->children->length;
 	}
 	size_t parent_height = parent->height - parent_offset;
 
@@ -136,10 +136,10 @@ static void apply_vert_layout(struct sway_container *parent) {
 	}
 	size_t parent_offset = 0;
 	if (parent->parent->layout == L_TABBED) {
-		parent_offset = config->font_height + 8;
+		parent_offset = container_titlebar_height();
 	} else if (parent->parent->layout == L_STACKED) {
-		parent_offset = (config->font_height + 8)
-			* parent->parent->children->length;
+		parent_offset =
+			container_titlebar_height() * parent->parent->children->length;
 	}
 	size_t parent_height = parent->height - parent_offset;
 
diff --git a/sway/tree/container.c b/sway/tree/container.c
index 5d88325f..3f30a079 100644
--- a/sway/tree/container.c
+++ b/sway/tree/container.c
@@ -510,7 +510,7 @@ static struct sway_container *container_at_tabbed(struct sway_container *parent,
 	struct sway_seat *seat = input_manager_current_seat(input_manager);
 
 	// Tab titles
-	int title_height = config->border_thickness * 2 + config->font_height;
+	int title_height = container_titlebar_height();
 	if (oy < parent->y + title_height) {
 		int tab_width = parent->width / parent->children->length;
 		int child_index = (ox - parent->x) / tab_width;
@@ -847,3 +847,7 @@ void container_notify_child_title_changed(struct sway_container *container) {
 	container_update_title_textures(container);
 	container_notify_child_title_changed(container->parent);
 }
+
+size_t container_titlebar_height() {
+	return config->font_height + TITLEBAR_V_PADDING * 2;
+}
diff --git a/sway/tree/view.c b/sway/tree/view.c
index 1280dc8d..07157818 100644
--- a/sway/tree/view.c
+++ b/sway/tree/view.c
@@ -182,10 +182,10 @@ void view_autoconfigure(struct sway_view *view) {
 	// area. We have to offset the surface y by the height of the title bar, and
 	// disable any top border because we'll always have the title bar.
 	if (view->swayc->parent->layout == L_TABBED) {
-		y_offset = config->font_height + 8;
+		y_offset = container_titlebar_height();
 		view->border_top = 0;
 	} else if (view->swayc->parent->layout == L_STACKED) {
-		y_offset = (config->font_height + 8)
+		y_offset = container_titlebar_height()
 			* view->swayc->parent->children->length;
 		view->border_top = 0;
 	}
@@ -218,8 +218,8 @@ void view_autoconfigure(struct sway_view *view) {
 			height = view->swayc->height - y_offset
 				- view->border_thickness * view->border_bottom;
 		} else {
-			y = view->swayc->y + config->font_height + 8;
-			height = view->swayc->height - config->font_height - 8
+			y = view->swayc->y + container_titlebar_height();
+			height = view->swayc->height - container_titlebar_height()
 				- view->border_thickness * view->border_bottom;
 		}
 		break;