From 9ac2342a673e899af87b8f8406820f2cc40bb185 Mon Sep 17 00:00:00 2001
From: mwenzkowski <29407878+mwenzkowski@users.noreply.github.com>
Date: Tue, 16 Apr 2019 21:20:48 +0200
Subject: [PATCH] Don't apply hide_edge_borders to floating windows

This change matches i3's behavior.
---
 sway/tree/view.c | 36 ++++++++++++++++++------------------
 1 file changed, 18 insertions(+), 18 deletions(-)

diff --git a/sway/tree/view.c b/sway/tree/view.c
index 40432661..e8f5a299 100644
--- a/sway/tree/view.c
+++ b/sway/tree/view.c
@@ -221,8 +221,10 @@ void view_autoconfigure(struct sway_view *view) {
 
 	con->border_top = con->border_bottom = true;
 	con->border_left = con->border_right = true;
+	double y_offset = 0;
+
+	if (!container_is_floating(con) && ws) {
 
-	if (ws) {
 		bool smart = config->hide_edge_borders == E_SMART ||
 			(config->hide_edge_borders == E_SMART_NO_GAPS &&
 			!gaps_to_edge(view));
@@ -240,24 +242,22 @@ void view_autoconfigure(struct sway_view *view) {
 			int bottom_y = con->y + con->height + con->current_gaps.bottom;
 			con->border_bottom = bottom_y != ws->y + ws->height;
 		}
-	}
 
-	double y_offset = 0;
-
-	// In a tabbed or stacked container, the container's y is the top of the
-	// title 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.
-	list_t *siblings = container_get_siblings(con);
-	bool show_titlebar = (siblings && siblings->length > 1)
-		|| !config->hide_lone_tab;
-	if (show_titlebar && !container_is_floating(con)) {
-		enum sway_container_layout layout = container_parent_layout(con);
-		if (layout == L_TABBED) {
-			y_offset = container_titlebar_height();
-			con->border_top = false;
-		} else if (layout == L_STACKED) {
-			y_offset = container_titlebar_height() * siblings->length;
-			con->border_top = false;
+		// In a tabbed or stacked container, the container's y is the top of the
+		// title 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.
+		list_t *siblings = container_get_siblings(con);
+		bool show_titlebar = (siblings && siblings->length > 1)
+			|| !config->hide_lone_tab;
+		if (show_titlebar) {
+			enum sway_container_layout layout = container_parent_layout(con);
+			if (layout == L_TABBED) {
+				y_offset = container_titlebar_height();
+				con->border_top = false;
+			} else if (layout == L_STACKED) {
+				y_offset = container_titlebar_height() * siblings->length;
+				con->border_top = false;
+			}
 		}
 	}