nixo
/
gri3-wm
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

cmd_focus_sibling: Fix crash on workspace level 

Fixes #3997
This commit is contained in:
Orestis Floros 2020-04-09 11:29:14 +02:00
parent 016d4a3f45
commit 4b4f1f604f
No known key found for this signature in database
GPG Key ID: A09DBD7D3222C1C3
2 changed files with 22 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/commands.c
View File

@ -1299,7 +1299,16 @@ void cmd_focus_sibling(I3_CMD, const char *direction_str) {
} }
Con *next = get_tree_next_sibling(current->con, direction); Con *next = get_tree_next_sibling(current->con, direction);
if (next) { if (next) {
con_activate(next); if (next->type == CT_WORKSPACE) {
/* On the workspace level, we need to make sure that the
* workspace change happens properly. However, workspace_show
* descends focus so we also have to put focus on the workspace
* itself to maintain consistency. See #3997. */
workspace_show(next);
con_focus(next);
} else {
con_activate(next);
}
} }
} }

12
testcases/t/307-focus-next-prev.t
View File

@ -69,4 +69,16 @@ cmp_tree(
cmd 'focus parent, focus next sibling'; cmd 'focus parent, focus next sibling';
}); });
# See #3997
cmd 'workspace 2';
open_window;
cmd 'workspace 1';
open_window;
cmd 'focus parent, focus parent, focus next sibling, focus prev sibling';
does_i3_live;
is(focused_ws, '1', 'Back and forth between workspaces');
cmd 'focus parent, focus parent, focus next sibling';
is(focused_ws, '2', "Workspace 2 focused with 'focus next sibling'");
done_testing; done_testing;