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

tree_move: Don't change focus order when swapping containers 

The call to ewmh_update_wm_desktop is removed since the change happens
under the same parent.
This commit is contained in:
Orestis Floros 2018-10-09 22:39:38 +03:00
parent 9380a75186
commit 11dd2dbc66
No known key found for this signature in database
GPG Key ID: E9AD9F32E401E38F
2 changed files with 25 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/move.c
View File

@ -298,9 +298,10 @@ void tree_move(Con *con, int direction) {
/* easy case: the move is within this container */
if (same_orientation == con->parent) {
DLOG("We are in the same container\n");
Con *swap;
if ((swap = (direction == D_LEFT || direction == D_UP ? TAILQ_PREV(con, nodes_head, nodes) : TAILQ_NEXT(con, nodes)))) {
Con *swap = (direction == D_LEFT || direction == D_UP)
? TAILQ_PREV(con, nodes_head, nodes)
: TAILQ_NEXT(con, nodes);
if (swap) {
if (!con_is_leaf(swap)) {
DLOG("Moving into our bordering branch\n");
target = con_descend_direction(swap, direction);
@ -312,17 +313,15 @@ void tree_move(Con *con, int direction) {
insert_con_into(con, target, position);
goto end;
}
if (direction == D_LEFT || direction == D_UP)
DLOG("Swapping with sibling.\n");
if (direction == D_LEFT || direction == D_UP) {
TAILQ_SWAP(swap, con, &(swap->parent->nodes_head), nodes);
else
} else {
TAILQ_SWAP(con, swap, &(swap->parent->nodes_head), nodes);
}
TAILQ_REMOVE(&(con->parent->focus_head), con, focused);
TAILQ_INSERT_HEAD(&(swap->parent->focus_head), con, focused);
DLOG("Swapped.\n");
ipc_send_window_event("move", con);
ewmh_update_wm_desktop();
return;
}

16
testcases/t/294-focus-order.t
View File

@ -125,6 +125,22 @@ $windows[0] = open_window;
cmd '[id=' . $windows[3]->id . '] move right';
confirm_focus('unfocused move from other output');
#####################################################################
# Test that moving an unfocused container inside its original parent
# maintains the correct focus order.
#####################################################################
fresh_workspace;
$windows[0] = open_window;
$windows[1] = open_window;
cmd 'split v';
$windows[2] = open_window;
$windows[3] = open_window;
focus_windows;
cmd '[id=' . $windows[2]->id . '] move up';
confirm_focus('split-v + unfocused move inside parent');
######################################################################
# Test that moving an unfocused container maintains the correct focus
# order.