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 (b14bd1b0b1) 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.
This commit is contained in:
Ryan Dwyer 2018-07-26 18:36:46 +10:00
parent a2164c6661
commit 08cfba2192
11 changed files with 126 additions and 86 deletions

View file

@ -297,6 +297,11 @@ void container_notify_subtree_changed(struct sway_container *container);
*/ */
size_t container_titlebar_height(void); size_t container_titlebar_height(void);
/**
* Resize and center the container in its workspace.
*/
void container_init_floating(struct sway_container *container);
void container_set_floating(struct sway_container *container, bool enable); void container_set_floating(struct sway_container *container, bool enable);
void container_set_geometry_from_floating_view(struct sway_container *con); void container_set_geometry_from_floating_view(struct sway_container *con);
@ -340,6 +345,12 @@ void container_end_mouse_operation(struct sway_container *container);
void container_set_fullscreen(struct sway_container *container, bool enable); void container_set_fullscreen(struct sway_container *container, bool enable);
/**
* Return true if the container is floating, or a child of a floating split
* container.
*/
bool container_is_floating_or_child(struct sway_container *container);
/** /**
* Return true if the container is fullscreen, or a child of a fullscreen split * Return true if the container is fullscreen, or a child of a fullscreen split
* container. * container.

View file

@ -234,11 +234,6 @@ void view_get_constraints(struct sway_view *view, double *min_width,
uint32_t view_configure(struct sway_view *view, double lx, double ly, int width, uint32_t view_configure(struct sway_view *view, double lx, double ly, int width,
int height); int height);
/**
* Center the view in its workspace and build the swayc decorations around it.
*/
void view_init_floating(struct sway_view *view);
/** /**
* Configure the view's position and size based on the swayc's position and * Configure the view's position and size based on the swayc's position and
* size, taking borders into consideration. * size, taking borders into consideration.

View file

@ -17,9 +17,16 @@ struct cmd_results *cmd_floating(int argc, char **argv) {
} }
struct sway_container *container = struct sway_container *container =
config->handler_context.current_container; config->handler_context.current_container;
if (container->type != C_VIEW) { if (container->type == C_WORKSPACE && container->children->length == 0) {
// TODO: This doesn't strictly speaking have to be true return cmd_results_new(CMD_INVALID, "floating",
return cmd_results_new(CMD_INVALID, "float", "Only views can float"); "Can't float an empty workspace");
}
if (container->type == C_WORKSPACE) {
// Wrap the workspace's children in a container so we can float it
struct sway_container *workspace = container;
container = container_wrap_children(container);
workspace->layout = L_HORIZ;
seat_set_focus(config->handler_context.seat, container);
} }
bool wants_floating; bool wants_floating;

View file

@ -298,9 +298,15 @@ static struct cmd_results *move_to_position(struct sway_container *container,
} }
static struct cmd_results *move_to_scratchpad(struct sway_container *con) { static struct cmd_results *move_to_scratchpad(struct sway_container *con) {
if (con->type != C_CONTAINER && con->type != C_VIEW) { if (con->type == C_WORKSPACE && con->children->length == 0) {
return cmd_results_new(CMD_INVALID, "move", return cmd_results_new(CMD_INVALID, "move",
"Only views and containers can be moved to the scratchpad"); "Can't move an empty workspace to the scratchpad");
}
if (con->type == C_WORKSPACE) {
// Wrap the workspace's children in a container
struct sway_container *workspace = con;
con = container_wrap_children(con);
workspace->layout = L_HORIZ;
} }
if (con->scratchpad) { if (con->scratchpad) {
return cmd_results_new(CMD_INVALID, "move", return cmd_results_new(CMD_INVALID, "move",

View file

@ -779,7 +779,7 @@ static void render_floating_container(struct sway_output *soutput,
} }
render_view(soutput, damage, con, colors); render_view(soutput, damage, con, colors);
} else { } else {
render_container(soutput, damage, con, false); render_container(soutput, damage, con, con->current.focused);
} }
} }

View file

@ -598,7 +598,10 @@ void dispatch_cursor_button(struct sway_cursor *cursor,
seat_set_focus_layer(cursor->seat, layer); seat_set_focus_layer(cursor->seat, layer);
} }
seat_pointer_notify_button(cursor->seat, time_msec, button, state); seat_pointer_notify_button(cursor->seat, time_msec, button, state);
} else if (cont && container_is_floating(cont)) { } else if (cont && container_is_floating_or_child(cont)) {
while (cont->parent->layout != L_FLOATING) {
cont = cont->parent;
}
dispatch_cursor_button_floating(cursor, time_msec, button, state, dispatch_cursor_button_floating(cursor, time_msec, button, state,
surface, sx, sy, cont); surface, sx, sy, cont);
} else if (surface && cont && cont->type != C_VIEW) { } else if (surface && cont && cont->type != C_VIEW) {

View file

@ -184,7 +184,6 @@ static void handle_seat_container_destroy(struct wl_listener *listener,
bool set_focus = bool set_focus =
focus != NULL && focus != NULL &&
(focus == con || container_has_child(con, focus)) && (focus == con || container_has_child(con, focus)) &&
con->parent && con->parent->children->length > 1 &&
con->type != C_WORKSPACE; con->type != C_WORKSPACE;
seat_container_destroy(seat_con); 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); seat->has_focus = (container != NULL);
update_debug_tree(); update_debug_tree();

View file

@ -72,11 +72,7 @@ static void scratchpad_show(struct sway_container *con) {
if (!wlr_box_contains_point(&workspace_box, center_lx, center_ly)) { if (!wlr_box_contains_point(&workspace_box, center_lx, center_ly)) {
// Maybe resize it // Maybe resize it
if (con->width > ws->width || con->height > ws->height) { if (con->width > ws->width || con->height > ws->height) {
// TODO: Do this properly once we can float C_CONTAINERs container_init_floating(con);
if (con->type == C_VIEW) {
view_init_floating(con->sway_view);
arrange_windows(con);
}
} }
// Center it // Center it
@ -85,6 +81,7 @@ static void scratchpad_show(struct sway_container *con) {
container_floating_move_to(con, new_lx, new_ly); container_floating_move_to(con, new_lx, new_ly);
} }
arrange_windows(ws);
seat_set_focus(seat, con); seat_set_focus(seat, con);
container_set_dirty(con->parent); container_set_dirty(con->parent);

View file

@ -407,6 +407,10 @@ struct sway_container *container_flatten(struct sway_container *container) {
* This function just wraps container_destroy_noreaping(), then does reaping. * This function just wraps container_destroy_noreaping(), then does reaping.
*/ */
struct sway_container *container_destroy(struct sway_container *con) { struct sway_container *container_destroy(struct sway_container *con) {
if (con->is_fullscreen) {
struct sway_container *ws = container_parent(con, C_WORKSPACE);
ws->sway_workspace->fullscreen = NULL;
}
struct sway_container *parent = container_destroy_noreaping(con); struct sway_container *parent = container_destroy_noreaping(con);
if (!parent) { if (!parent) {
@ -945,23 +949,81 @@ size_t container_titlebar_height() {
return config->font_height + TITLEBAR_V_PADDING * 2; return config->font_height + TITLEBAR_V_PADDING * 2;
} }
void container_init_floating(struct sway_container *con) {
if (!sway_assert(con->type == C_VIEW || con->type == C_CONTAINER,
"Expected a view or container")) {
return;
}
struct sway_container *ws = container_parent(con, C_WORKSPACE);
int min_width, min_height;
int max_width, max_height;
if (config->floating_minimum_width == -1) { // no minimum
min_width = 0;
} else if (config->floating_minimum_width == 0) { // automatic
min_width = 75;
} else {
min_width = config->floating_minimum_width;
}
if (config->floating_minimum_height == -1) { // no minimum
min_height = 0;
} else if (config->floating_minimum_height == 0) { // automatic
min_height = 50;
} else {
min_height = config->floating_minimum_height;
}
if (config->floating_maximum_width == -1) { // no maximum
max_width = INT_MAX;
} else if (config->floating_maximum_width == 0) { // automatic
max_width = ws->width * 0.6666;
} else {
max_width = config->floating_maximum_width;
}
if (config->floating_maximum_height == -1) { // no maximum
max_height = INT_MAX;
} else if (config->floating_maximum_height == 0) { // automatic
max_height = ws->height * 0.6666;
} else {
max_height = config->floating_maximum_height;
}
if (con->type == C_CONTAINER) {
con->width = max_width;
con->height = max_height;
con->x = ws->x + (ws->width - con->width) / 2;
con->y = ws->y + (ws->height - con->height) / 2;
} else {
struct sway_view *view = con->sway_view;
view->width = fmax(min_width, fmin(view->natural_width, max_width));
view->height = fmax(min_height, fmin(view->natural_height, max_height));
view->x = ws->x + (ws->width - view->width) / 2;
view->y = ws->y + (ws->height - view->height) / 2;
// If the view's border is B_NONE then these properties are ignored.
view->border_top = view->border_bottom = true;
view->border_left = view->border_right = true;
container_set_geometry_from_floating_view(view->swayc);
}
}
void container_set_floating(struct sway_container *container, bool enable) { void container_set_floating(struct sway_container *container, bool enable) {
if (container_is_floating(container) == enable) { if (container_is_floating(container) == enable) {
return; return;
} }
struct sway_container *workspace = container_parent(container, C_WORKSPACE); struct sway_container *workspace = container_parent(container, C_WORKSPACE);
struct sway_seat *seat = input_manager_current_seat(input_manager);
if (enable) { if (enable) {
container_remove_child(container); container_remove_child(container);
container_add_child(workspace->sway_workspace->floating, container); container_add_child(workspace->sway_workspace->floating, container);
container_init_floating(container);
if (container->type == C_VIEW) { if (container->type == C_VIEW) {
view_init_floating(container->sway_view);
view_set_tiled(container->sway_view, false); view_set_tiled(container->sway_view, false);
} }
seat_set_focus(seat, seat_get_focus_inactive(seat, container));
container_reap_empty_recursive(workspace);
} else { } else {
// Returning to tiled // Returning to tiled
if (container->scratchpad) { if (container->scratchpad) {
@ -975,7 +1037,6 @@ void container_set_floating(struct sway_container *container, bool enable) {
view_set_tiled(container->sway_view, true); view_set_tiled(container->sway_view, true);
} }
container->is_sticky = false; container->is_sticky = false;
container_reap_empty_recursive(workspace->sway_workspace->floating);
} }
container_end_mouse_operation(container); container_end_mouse_operation(container);
@ -1195,6 +1256,17 @@ void container_set_fullscreen(struct sway_container *container, bool enable) {
ipc_event_window(container, "fullscreen_mode"); ipc_event_window(container, "fullscreen_mode");
} }
bool container_is_floating_or_child(struct sway_container *container) {
do {
if (container->parent && container->parent->layout == L_FLOATING) {
return true;
}
container = container->parent;
} while (container && container->type != C_WORKSPACE);
return false;
}
bool container_is_fullscreen_or_child(struct sway_container *container) { bool container_is_fullscreen_or_child(struct sway_container *container) {
do { do {
if (container->is_fullscreen) { if (container->is_fullscreen) {

View file

@ -387,9 +387,11 @@ void container_move(struct sway_container *container,
// If moving a fullscreen view, only consider outputs // If moving a fullscreen view, only consider outputs
if (container->is_fullscreen) { if (container->is_fullscreen) {
current = container_parent(container, C_OUTPUT); current = container_parent(container, C_OUTPUT);
} else if (container_is_fullscreen_or_child(container)) { } else if (container_is_fullscreen_or_child(container) ||
container_is_floating_or_child(container)) {
// If we've fullscreened a split container, only allow the child to move // If we've fullscreened a split container, only allow the child to move
// around within the fullscreen parent. // around within the fullscreen parent.
// Same with floating a split container.
struct sway_container *ws = container_parent(container, C_WORKSPACE); struct sway_container *ws = container_parent(container, C_WORKSPACE);
top = ws->sway_workspace->fullscreen; top = ws->sway_workspace->fullscreen;
} }
@ -465,6 +467,9 @@ void container_move(struct sway_container *container,
if ((index == parent->children->length - 1 && offs > 0) if ((index == parent->children->length - 1 && offs > 0)
|| (index == 0 && offs < 0)) { || (index == 0 && offs < 0)) {
if (current->parent == container->parent) { if (current->parent == container->parent) {
if (parent->parent->layout == L_FLOATING) {
return;
}
if (!parent->is_fullscreen && if (!parent->is_fullscreen &&
(parent->layout == L_TABBED || (parent->layout == L_TABBED ||
parent->layout == L_STACKED)) { parent->layout == L_STACKED)) {
@ -488,10 +493,14 @@ void container_move(struct sway_container *container,
sibling = parent->children->items[index + offs]; sibling = parent->children->items[index + offs];
wlr_log(WLR_DEBUG, "Selecting sibling id:%zd", sibling->id); wlr_log(WLR_DEBUG, "Selecting sibling id:%zd", sibling->id);
} }
} else if (!parent->is_fullscreen && (parent->layout == L_TABBED || } else if (!parent->is_fullscreen &&
parent->parent->layout != L_FLOATING &&
(parent->layout == L_TABBED ||
parent->layout == L_STACKED)) { parent->layout == L_STACKED)) {
move_out_of_tabs_stacks(container, current, move_dir, offs); move_out_of_tabs_stacks(container, current, move_dir, offs);
return; return;
} else if (parent->parent->layout == L_FLOATING) {
return;
} else { } else {
wlr_log(WLR_DEBUG, "Moving up to find a parallel container"); wlr_log(WLR_DEBUG, "Moving up to find a parallel container");
current = current->parent; current = current->parent;
@ -717,10 +726,6 @@ struct sway_container *container_get_in_direction(
enum movement_direction dir) { enum movement_direction dir) {
struct sway_container *parent = container->parent; struct sway_container *parent = container->parent;
if (container_is_floating(container)) {
return NULL;
}
if (dir == MOVE_CHILD) { if (dir == MOVE_CHILD) {
return seat_get_focus_inactive(seat, container); return seat_get_focus_inactive(seat, container);
} }
@ -732,7 +737,7 @@ struct sway_container *container_get_in_direction(
parent = container->parent; parent = container->parent;
} else { } else {
if (dir == MOVE_PARENT) { if (dir == MOVE_PARENT) {
if (parent->type == C_OUTPUT) { if (parent->type == C_OUTPUT || container_is_floating(container)) {
return NULL; return NULL;
} else { } else {
return parent; return parent;

View file

@ -168,55 +168,6 @@ uint32_t view_configure(struct sway_view *view, double lx, double ly, int width,
return 0; return 0;
} }
void view_init_floating(struct sway_view *view) {
struct sway_container *ws = container_parent(view->swayc, C_WORKSPACE);
int min_width, min_height;
int max_width, max_height;
if (config->floating_minimum_width == -1) { // no minimum
min_width = 0;
} else if (config->floating_minimum_width == 0) { // automatic
min_width = 75;
} else {
min_width = config->floating_minimum_width;
}
if (config->floating_minimum_height == -1) { // no minimum
min_height = 0;
} else if (config->floating_minimum_height == 0) { // automatic
min_height = 50;
} else {
min_height = config->floating_minimum_height;
}
if (config->floating_maximum_width == -1) { // no maximum
max_width = INT_MAX;
} else if (config->floating_maximum_width == 0) { // automatic
max_width = ws->width * 0.6666;
} else {
max_width = config->floating_maximum_width;
}
if (config->floating_maximum_height == -1) { // no maximum
max_height = INT_MAX;
} else if (config->floating_maximum_height == 0) { // automatic
max_height = ws->height * 0.6666;
} else {
max_height = config->floating_maximum_height;
}
view->width = fmax(min_width, fmin(view->natural_width, max_width));
view->height = fmax(min_height, fmin(view->natural_height, max_height));
view->x = ws->x + (ws->width - view->width) / 2;
view->y = ws->y + (ws->height - view->height) / 2;
// If the view's border is B_NONE then these properties are ignored.
view->border_top = view->border_bottom = true;
view->border_left = view->border_right = true;
container_set_geometry_from_floating_view(view->swayc);
}
void view_autoconfigure(struct sway_view *view) { void view_autoconfigure(struct sway_view *view) {
if (!sway_assert(view->swayc, if (!sway_assert(view->swayc,
"Called view_autoconfigure() on a view without a swayc")) { "Called view_autoconfigure() on a view without a swayc")) {
@ -626,10 +577,8 @@ void view_unmap(struct sway_view *view) {
struct sway_container *ws = container_parent(view->swayc, C_WORKSPACE); struct sway_container *ws = container_parent(view->swayc, C_WORKSPACE);
struct sway_container *parent; struct sway_container *parent;
if (view->swayc->is_fullscreen) { if (container_is_fullscreen_or_child(view->swayc)) {
ws->sway_workspace->fullscreen = NULL;
parent = container_destroy(view->swayc); parent = container_destroy(view->swayc);
arrange_windows(ws->parent); arrange_windows(ws->parent);
} else { } else {
parent = container_destroy(view->swayc); parent = container_destroy(view->swayc);