From 4762bcb3b9a1b9043ce78c7765d4513b0c5da635 Mon Sep 17 00:00:00 2001
From: "D.B" <thejan.2009@gmail.com>
Date: Sun, 23 Oct 2016 13:19:02 +0200
Subject: [PATCH] wrap some views under workspaces

If workspace layout is set to tabbed or stacked, its C_VIEW children
should get wrapped in a container. Alongside that, move_container was
modified to retain previous functionality.
---
 sway/layout.c | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/sway/layout.c b/sway/layout.c
index b37b82da6..ea4a680d7 100644
--- a/sway/layout.c
+++ b/sway/layout.c
@@ -66,6 +66,9 @@ void add_child(swayc_t *parent, swayc_t *child) {
 	if (!parent->focused) {
 		parent->focused = child;
 	}
+	if (parent->type == C_WORKSPACE && child->type == C_VIEW && (parent->workspace_layout == L_TABBED || parent->workspace_layout == L_STACKED)) {
+		child = new_container(child, parent->workspace_layout);
+	}
 }
 
 void insert_child(swayc_t *parent, swayc_t *child, int index) {
@@ -80,6 +83,10 @@ void insert_child(swayc_t *parent, swayc_t *child, int index) {
 	if (!parent->focused) {
 		parent->focused = child;
 	}
+	if (parent->type == C_WORKSPACE && child->type == C_VIEW && (parent->workspace_layout == L_TABBED || parent->workspace_layout == L_STACKED)) {
+		child = new_container(child, parent->workspace_layout);
+	}
+
 }
 
 void add_floating(swayc_t *ws, swayc_t *child) {
@@ -255,6 +262,19 @@ void move_container(swayc_t *container, enum movement_direction dir) {
 	swayc_t *parent = container->parent;
 	swayc_t *child = container;
 	bool ascended = false;
+
+	// View is wrapped in intermediate container which is needed for displaying
+	// the titlebar. Moving only the view outside of its parent container would just
+	// wrap it again under worspace. There would effectively be no movement,
+	// just a change of wrapping container.
+	if (child->type == C_VIEW &&
+		parent->type == C_CONTAINER &&
+		parent->children->length == 1 &&
+		parent->parent->type == C_WORKSPACE) {
+		child = parent;
+		parent = parent->parent;
+	}
+
 	while (true) {
 		sway_log(L_DEBUG, "container:%p, parent:%p, child %p,",
 				container,parent,child);
@@ -348,6 +368,9 @@ void move_container(swayc_t *container, enum movement_direction dir) {
 			}
 			// Create container around workspace to insert child into
 			parent = new_container(parent, layout);
+			// Previous line set the resulting container's layout to
+			// workspace_layout. It should have been just layout.
+			parent->layout = parent->parent->layout;
 		}
 		ascended = true;
 		child = parent;