From: Michael Stapelberg <michael@stapelberg.de>
Date: Thu, 10 Nov 2011 19:35:36 +0000 (+0000)
Subject: Bugfix: Fix tabbed container resizing check (Thanks julien)
X-Git-Tag: 4.1~11
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=df095f9d68b3bd68bb30fe77b0b176ed16dd3252;p=i3%2Fi3

Bugfix: Fix tabbed container resizing check (Thanks julien)

Fixes: #534

Turns out that my check was wrong. We actually just need to make sure we are
dealing with the split container.
---

diff --git a/src/click.c b/src/click.c
index d0adc231..d3f9da49 100644
--- a/src/click.c
+++ b/src/click.c
@@ -127,13 +127,10 @@ static bool tiling_resize(Con *con, const xcb_button_press_event_t *event, const
          * window. Decorations will end up next to each other and the user
          * expects to switch to a window by clicking on its decoration. */
 
-        /* To make the check actually work in case of a h-split container, we
-         * need to find the resize con in this function (as opposed to in
-         * tiling_resize_for_border()). The con which was passed is the actualy
-         * child window, not the split container. */
-        while (con->type != CT_WORKSPACE &&
-               con->type != CT_FLOATING_CON &&
-               con->parent->orientation != VERT)
+        /* Since the container might either be the child *or* already a split
+         * container (in the case of a nested split container), we need to make
+         * sure that we are dealing with the split container here. */
+        if (con_is_leaf(con))
             con = con->parent;
 
         if ((con->layout == L_STACKED ||