From: Michael Stapelberg <michael@stapelberg.de>
Date: Mon, 18 Apr 2011 16:44:18 +0000 (+0200)
Subject: Bugfix: When moving floating cons to other workspaces, attach them to the workspace
X-Git-Tag: tree-pr3~77
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=36583ec6ee7ada36764a492fdaae158a5cdd04c3;p=i3%2Fi3

Bugfix: When moving floating cons to other workspaces, attach them to the workspace
---

diff --git a/src/con.c b/src/con.c
index 806d0b29..1adfcf35 100644
--- a/src/con.c
+++ b/src/con.c
@@ -549,8 +549,10 @@ void con_move_to_workspace(Con *con, Con *workspace) {
     Con *next = con_descend_focused(workspace);
 
     /* 3: we go up one level, but only when next is a normal container */
-    if (next->type != CT_WORKSPACE)
+    if (next->type != CT_WORKSPACE) {
+        DLOG("next originally = %p / %s / type %d\n", next, next->name, next->type);
         next = next->parent;
+    }
 
     /* 4: if the target container is floating, we get the workspace instead.
      * Only tiling windows need to get inserted next to the current container.
@@ -561,6 +563,12 @@ void con_move_to_workspace(Con *con, Con *workspace) {
         next = floatingcon->parent;
     }
 
+    if (con->type == CT_FLOATING_CON) {
+        Con *ws = con_get_workspace(next);
+        DLOG("This is a floating window, using workspace %p / %s\n", ws, ws->name);
+        next = ws;
+    }
+
     DLOG("Re-attaching container to %p / %s\n", next, next->name);
     /* 5: re-attach the con to the parent of this focused container */
     Con *parent = con->parent;
@@ -581,7 +589,7 @@ void con_move_to_workspace(Con *con, Con *workspace) {
      * don’t want to focus invisible workspaces */
     if (source_output != dest_output &&
         workspace_is_visible(workspace)) {
-        DLOG("Moved to a different output, focusing target");
+        DLOG("Moved to a different output, focusing target\n");
     } else {
         con_focus(focus_next);
     }