From: Michael Stapelberg <michael@stapelberg.de>
Date: Sun, 28 Nov 2010 17:05:53 +0000 (+0100)
Subject: Bugfix: Fix focus when moving Cons
X-Git-Tag: tree-pr1~32
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=4fcd2f6e7ef71bef62044b09993e4dee88044e84;p=i3%2Fi3

Bugfix: Fix focus when moving Cons

When having two v-splits on a horizontal desktop:
----------------
|  t1   |  t3  |
|-------|------|
|  t2   |  t4  |
----------------
…focus is on t2, and you move it into the right v-split (move after h), the
focus was not properly updated. That is, inside the right v-split, focus was
correct, but the workspace focus was still pointing to the left v-split.
---

diff --git a/src/tree.c b/src/tree.c
index 2d8a1333..71d46362 100644
--- a/src/tree.c
+++ b/src/tree.c
@@ -496,6 +496,11 @@ void tree_move(char way, orientation_t orientation) {
         /* TODO: don’t influence focus handling? */
     }
 
+    /* We need to call con_focus() to fix the focus stack "above" the container
+     * we just inserted the focused container into (otherwise, the parent
+     * container(s) would still point to the old container(s)). */
+    con_focus(focused);
+
     if (con_num_children(old_parent) == 0) {
         DLOG("Old container empty after moving. Let's close it\n");
         tree_close(old_parent, false, false);