Bugfix: Fix restart with for_window floating configuration (Thanks garga)

Fixes: #668
Calling tree_close with dont_kill_parent=true will avoid it from closing the
workspace if it’s empty (and it’s temporarily empty, because 'floating disable'
detaches, then re-attaches the window).

diff --git a/src/floating.c b/src/floating.c
index 0637521fbce020630c22ec98bd8f2efffee3785a..96c0972403730d909874e04db7ff217e4d015e9e 100644 (file)
--- a/src/floating.c
+++ b/src/floating.c
@@ -223,7 +223,7 @@ void floating_disable(Con *con, bool automatic) {
     /* 2: kill parent container */
     TAILQ_REMOVE(&(con->parent->parent->floating_head), con->parent, floating_windows);
     TAILQ_REMOVE(&(con->parent->parent->focus_head), con->parent, focused);
-    tree_close(con->parent, DONT_KILL_WINDOW, false, false);
+    tree_close(con->parent, DONT_KILL_WINDOW, true, false);
 
     /* 3: re-attach to the parent of the currently focused con on the workspace
      * this floating con was on */