Allow containers to float

Things worth noting:

* When a fullscreen view unmaps, the check to unset fullscreen on the
workspace has been moved out of view_unmap and into container_destroy,
because containers can be fullscreen too
* The calls to `container_reap_empty_recursive(workspace)` have been
removed from `container_set_floating`. That function reaps upwards so it
wouldn't do anything. I'm probably the one who originally added it...
* My fix (b14bd1b0b1536039e4f46fe94515c7c44e7afc61) for the tabbed child
crash has a side effect where when you close a floating container, focus
is not given to the tiled container again. I've removed my fix and
removed the call to `send_cursor_motion` from `seat_set_focus_warp`. We
should consider calling it from somewhere earlier in the call stack.

1 files changed, 0 insertions, 5 deletions

diff --git a/sway/input/seat.c b/sway/input/seat.c
index e7b6e0c5..877a93c6 100644
--- a/sway/input/seat.c
+++ b/sway/input/seat.c
@@ -184,7 +184,6 @@ static void handle_seat_container_destroy(struct wl_listener *listener,
         bool set_focus =
                 focus != NULL &&
                 (focus == con || container_has_child(con, focus)) &&
-                con->parent && con->parent->children->length > 1 &&
                 con->type != C_WORKSPACE;
 
         seat_container_destroy(seat_con);
@@ -754,10 +753,6 @@ void seat_set_focus_warp(struct sway_seat *seat,
                 }
         }
 
-        if (last_focus != NULL) {
-                cursor_send_pointer_motion(seat->cursor, 0, true);
-        }
-
         seat->has_focus = (container != NULL);
 
         update_debug_tree();