Bugfix: Only change focus in tree_close() when the container was focused before

Fixes: #441

I’m not sure if that’s really all there is to it. Seems a bit too simple :).
We’ll see if I missed some corner cases in the next few days…
next
Michael Stapelberg 2011-09-18 17:43:02 +01:00
parent 1f95fbfa3e
commit 2fe0949abd
1 changed files with 3 additions and 2 deletions

View File

@ -220,12 +220,13 @@ bool tree_close(Con *con, kill_window_t kill_window, bool dont_kill_parent) {
if (was_mapped || con == focused) {
if ((kill_window != DONT_KILL_WINDOW) || !dont_kill_parent || con == focused) {
DLOG("focusing %p / %s\n", next, next->name);
/* TODO: check if the container (or one of its children) was focused */
if (next->type == CT_DOCKAREA) {
/* Instead of focusing the dockarea, we need to restore focus to the workspace */
con_focus(con_descend_focused(output_get_content(next->parent)));
} else {
con_focus(next);
if (con != focused)
DLOG("not changing focus, the container was not focused before\n");
else con_focus(next);
}
}
else {