Bugfix: Insert container at the correct position on workspace-level when workspace_la...

Fixes #533

The problem was that the code was always executed. While it *attaches* the new
container to the workspace container, it also sets current = NULL and thus
always appends the container instead of inserting it after the currently
focused child. So now, we just don’t execute that code at all for
workspace_layout == default.

diff --git a/src/con.c b/src/con.c
index 0460568944574bbbd2fbd0e3a9eb3e36e9644212..ee84467b82da4dea7b2becdc5c061d9e8d0e7093 100644 (file)
--- a/src/con.c
+++ b/src/con.c
@@ -129,7 +129,9 @@ void con_attach(Con *con, Con *parent, bool ignore_focus) {
          * workspace or a new split container with the configured
          * workspace_layout).
          */
-        if (con->window != NULL && parent->type == CT_WORKSPACE) {
+        if (con->window != NULL &&
+            parent->type == CT_WORKSPACE &&
+            config.default_layout != L_DEFAULT) {
             DLOG("Parent is a workspace. Applying default layout...\n");
             Con *target = workspace_attach_to(parent);