From 7f558ce894934bd347140f5edd6419bf7d115c26 Mon Sep 17 00:00:00 2001 From: "D.B" Date: Fri, 7 Oct 2016 07:10:06 +0200 Subject: wrap container in direction (if possible) Introduces container wrapping - if there is no other viable move, the selection wraps on the first container where such action is possible. --- sway/layout.c | 22 ++++++++++++++++++---- 1 file changed, 18 insertions(+), 4 deletions(-) (limited to 'sway/layout.c') diff --git a/sway/layout.c b/sway/layout.c index 6012af66..f10e21bd 100644 --- a/sway/layout.c +++ b/sway/layout.c @@ -1084,13 +1084,17 @@ swayc_t *get_swayc_in_direction_under(swayc_t *container, enum movement_directio return container->fullscreen; } + swayc_t *wrap_candidate = NULL; while (true) { // Test if we can even make a difference here bool can_move = false; int diff = 0; if (parent->type == C_ROOT) { - sway_log(L_DEBUG, "Moving between outputs"); swayc_t *output = swayc_adjacent_output(container, dir, &abs_pos, true); + if (!output || output == container) { + return wrap_candidate; + } + sway_log(L_DEBUG, "Moving between outputs"); return get_swayc_in_output_direction(output, dir); } else { if (dir == MOVE_LEFT || dir == MOVE_RIGHT) { @@ -1108,8 +1112,18 @@ swayc_t *get_swayc_in_direction_under(swayc_t *container, enum movement_directio if (can_move) { int desired = index_child(container) + diff; - if (container->is_floating || desired < 0 || desired >= parent->children->length) { + if (container->is_floating) { can_move = false; + } else if (desired < 0 || desired >= parent->children->length) { + can_move = false; + int len = parent->children->length; + if (!wrap_candidate && len > 1) { + if (desired < 0) { + wrap_candidate = parent->children->items[len-1]; + } else { + wrap_candidate = parent->children->items[0]; + } + } } else { return parent->children->items[desired]; } @@ -1118,8 +1132,8 @@ swayc_t *get_swayc_in_direction_under(swayc_t *container, enum movement_directio container = parent; parent = parent->parent; if (!parent || container == limit) { - // Nothing we can do - return NULL; + // wrapping is the last chance + return wrap_candidate; } } } -- cgit v1.2.3 From d3f5ac8cbbdde88e5822c1d9dd20ce9cbde8cbbe Mon Sep 17 00:00:00 2001 From: "D.B" Date: Fri, 7 Oct 2016 07:32:57 +0200 Subject: add force_focus_wrapping option --- sway/layout.c | 3 +++ 1 file changed, 3 insertions(+) (limited to 'sway/layout.c') diff --git a/sway/layout.c b/sway/layout.c index f10e21bd..a7206e9a 100644 --- a/sway/layout.c +++ b/sway/layout.c @@ -1123,6 +1123,9 @@ swayc_t *get_swayc_in_direction_under(swayc_t *container, enum movement_directio } else { wrap_candidate = parent->children->items[0]; } + if (config->force_focus_wrapping) { + return wrap_candidate; + } } } else { return parent->children->items[desired]; -- cgit v1.2.3 From 379b1a0378292ff1c470276f1caad59f36c7fc66 Mon Sep 17 00:00:00 2001 From: "D.B" Date: Fri, 7 Oct 2016 07:42:51 +0200 Subject: add focus changing between floating containers It will also wrap if needed. --- sway/layout.c | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) (limited to 'sway/layout.c') diff --git a/sway/layout.c b/sway/layout.c index a7206e9a..3629df24 100644 --- a/sway/layout.c +++ b/sway/layout.c @@ -1113,7 +1113,17 @@ swayc_t *get_swayc_in_direction_under(swayc_t *container, enum movement_directio if (can_move) { int desired = index_child(container) + diff; if (container->is_floating) { - can_move = false; + if (desired < 0) { + wrap_candidate = parent->floating->items[parent->floating->length-1]; + } else if (desired >= parent->floating->length){ + wrap_candidate = parent->floating->items[0]; + } else { + wrap_candidate = parent->floating->items[desired]; + } + if (wrap_candidate) { + wlc_view_bring_to_front(wrap_candidate->handle); + } + return wrap_candidate; } else if (desired < 0 || desired >= parent->children->length) { can_move = false; int len = parent->children->length; -- cgit v1.2.3