Merge pull request #945 from thejan2009/workspace_layout

Fixes dealing with workspace_layout and related bugs [rfc]
This commit is contained in:
Drew DeVault 2016-10-11 08:24:38 -04:00 committed by GitHub
commit ce713efcd2
7 changed files with 44 additions and 26 deletions

View file

@ -68,6 +68,7 @@ struct sway_container {
enum swayc_types type; enum swayc_types type;
enum swayc_layouts layout; enum swayc_layouts layout;
enum swayc_layouts prev_layout; enum swayc_layouts prev_layout;
enum swayc_layouts workspace_layout;
/** /**
* Width and height of this container, without borders or gaps. * Width and height of this container, without borders or gaps.
@ -321,4 +322,10 @@ void update_visibility(swayc_t *container);
*/ */
void close_views(swayc_t *container); void close_views(swayc_t *container);
/**
* Assign layout to a container. Needed due to workspace container specifics.
* Workspace always needs L_HORIZ layout.
*/
swayc_t *swayc_change_layout(swayc_t *container, enum swayc_layouts layout);
#endif #endif

View file

@ -22,10 +22,10 @@ struct cmd_results *cmd_layout(int argc, char **argv) {
enum swayc_layouts old_layout = parent->layout; enum swayc_layouts old_layout = parent->layout;
if (strcasecmp(argv[0], "default") == 0) { if (strcasecmp(argv[0], "default") == 0) {
parent->layout = parent->prev_layout; swayc_change_layout(parent, parent->prev_layout);
if (parent->layout == L_NONE) { if (parent->layout == L_NONE) {
swayc_t *output = swayc_parent_by_type(parent, C_OUTPUT); swayc_t *output = swayc_parent_by_type(parent, C_OUTPUT);
parent->layout = default_layout(output); swayc_change_layout(parent, default_layout(output));
} }
} else { } else {
if (parent->layout != L_TABBED && parent->layout != L_STACKED) { if (parent->layout != L_TABBED && parent->layout != L_STACKED) {
@ -37,22 +37,22 @@ struct cmd_results *cmd_layout(int argc, char **argv) {
parent = new_container(parent, L_TABBED); parent = new_container(parent, L_TABBED);
} }
parent->layout = L_TABBED; swayc_change_layout(parent, L_TABBED);
} else if (strcasecmp(argv[0], "stacking") == 0) { } else if (strcasecmp(argv[0], "stacking") == 0) {
if (parent->type != C_CONTAINER && !swayc_is_empty_workspace(parent)) { if (parent->type != C_CONTAINER && !swayc_is_empty_workspace(parent)) {
parent = new_container(parent, L_STACKED); parent = new_container(parent, L_STACKED);
} }
parent->layout = L_STACKED; swayc_change_layout(parent, L_STACKED);
} else if (strcasecmp(argv[0], "splith") == 0) { } else if (strcasecmp(argv[0], "splith") == 0) {
parent->layout = L_HORIZ; swayc_change_layout(parent, L_HORIZ);
} else if (strcasecmp(argv[0], "splitv") == 0) { } else if (strcasecmp(argv[0], "splitv") == 0) {
parent->layout = L_VERT; swayc_change_layout(parent, L_VERT);
} else if (strcasecmp(argv[0], "toggle") == 0 && argc == 2 && strcasecmp(argv[1], "split") == 0) { } else if (strcasecmp(argv[0], "toggle") == 0 && argc == 2 && strcasecmp(argv[1], "split") == 0) {
if (parent->layout == L_VERT) { if (parent->layout == L_VERT) {
parent->layout = L_HORIZ; swayc_change_layout(parent, L_HORIZ);
} else { } else {
parent->layout = L_VERT; swayc_change_layout(parent, L_VERT);
} }
} }
} }

View file

@ -37,7 +37,7 @@ struct cmd_results *cmd_move(int argc, char **argv) {
if (!view->children || view->children->length == 0) { if (!view->children || view->children->length == 0) {
return cmd_results_new(CMD_FAILURE, "move", "Cannot move an empty workspace"); return cmd_results_new(CMD_FAILURE, "move", "Cannot move an empty workspace");
} }
view = new_container(view, view->layout); view = new_container(view, view->workspace_layout);
} if (view->type != C_CONTAINER && view->type != C_VIEW) { } if (view->type != C_CONTAINER && view->type != C_VIEW) {
return cmd_results_new(CMD_FAILURE, "move", "Can only move containers and views."); return cmd_results_new(CMD_FAILURE, "move", "Can only move containers and views.");
} }
@ -65,7 +65,7 @@ struct cmd_results *cmd_move(int argc, char **argv) {
if (!view->children || view->children->length == 0) { if (!view->children || view->children->length == 0) {
return cmd_results_new(CMD_FAILURE, "move", "Cannot move an empty workspace"); return cmd_results_new(CMD_FAILURE, "move", "Cannot move an empty workspace");
} }
view = new_container(view, view->layout); view = new_container(view, view->workspace_layout);
} else if (view->type != C_CONTAINER && view->type != C_VIEW) { } else if (view->type != C_CONTAINER && view->type != C_VIEW) {
return cmd_results_new(CMD_FAILURE, "move", "Can only move containers and views."); return cmd_results_new(CMD_FAILURE, "move", "Can only move containers and views.");
} else if (!(output = output_by_name(argv[3], &abs_pos))) { } else if (!(output = output_by_name(argv[3], &abs_pos))) {

View file

@ -25,11 +25,11 @@ static struct cmd_results *_do_split(int argc, char **argv, int layout) {
/* Case that focus is on an workspace with 0/1 children.change its layout */ /* Case that focus is on an workspace with 0/1 children.change its layout */
if (focused->type == C_WORKSPACE && focused->children->length <= 1) { if (focused->type == C_WORKSPACE && focused->children->length <= 1) {
sway_log(L_DEBUG, "changing workspace layout"); sway_log(L_DEBUG, "changing workspace layout");
focused->layout = layout; swayc_change_layout(focused, layout);
} else if (focused->type != C_WORKSPACE && focused->parent->children->length == 1) { } else if (focused->type != C_WORKSPACE && focused->parent->children->length == 1) {
/* Case of no siblings. change parent layout */ /* Case of no siblings. change parent layout */
sway_log(L_DEBUG, "changing container layout"); sway_log(L_DEBUG, "changing container layout");
focused->parent->layout = layout; swayc_change_layout(focused->parent, layout);
} else { } else {
/* regular case where new split container is build around focused container /* regular case where new split container is build around focused container
* or in case of workspace, container inherits its children */ * or in case of workspace, container inherits its children */

View file

@ -27,6 +27,7 @@ static swayc_t *new_swayc(enum swayc_types type) {
c->handle = -1; c->handle = -1;
c->gaps = -1; c->gaps = -1;
c->layout = L_NONE; c->layout = L_NONE;
c->workspace_layout = L_NONE;
c->type = type; c->type = type;
if (type != C_VIEW) { if (type != C_VIEW) {
c->children = create_list(); c->children = create_list();
@ -209,7 +210,8 @@ swayc_t *new_workspace(swayc_t *output, const char *name) {
swayc_t *workspace = new_swayc(C_WORKSPACE); swayc_t *workspace = new_swayc(C_WORKSPACE);
workspace->prev_layout = L_NONE; workspace->prev_layout = L_NONE;
workspace->layout = default_layout(output); workspace->layout = L_HORIZ;
workspace->workspace_layout = default_layout(output);
workspace->x = output->x; workspace->x = output->x;
workspace->y = output->y; workspace->y = output->y;
@ -262,7 +264,7 @@ swayc_t *new_container(swayc_t *child, enum swayc_layouts layout) {
// add container to workspace chidren // add container to workspace chidren
add_child(workspace, cont); add_child(workspace, cont);
// give them proper layouts // give them proper layouts
cont->layout = workspace->layout; cont->layout = workspace->workspace_layout;
cont->prev_layout = workspace->prev_layout; cont->prev_layout = workspace->prev_layout;
/* TODO: might break shit in move_container!!! workspace->layout = layout; */ /* TODO: might break shit in move_container!!! workspace->layout = layout; */
set_focused_container_for(workspace, get_focused_view(workspace)); set_focused_container_for(workspace, get_focused_view(workspace));
@ -944,3 +946,12 @@ swayc_t *swayc_tabbed_stacked_parent(swayc_t *con) {
} }
return NULL; return NULL;
} }
swayc_t *swayc_change_layout(swayc_t *container, enum swayc_layouts layout) {
if (container->type == C_WORKSPACE) {
container->workspace_layout = layout;
} else {
container->layout = layout;
}
return container;
}

View file

@ -398,17 +398,6 @@ static bool handle_view_created(wlc_handle handle) {
if (workspace && workspace->fullscreen) { if (workspace && workspace->fullscreen) {
set_focused_container(workspace->fullscreen); set_focused_container(workspace->fullscreen);
} }
// if parent container is a workspace, newview its only child and
// layout is tabbed/stacked, add a container around newview
swayc_t *parent_container = newview->parent;
if (parent_container && parent_container->type == C_WORKSPACE &&
parent_container->children && parent_container->children->length == 1 &&
(parent_container->layout == L_TABBED || parent_container->layout == L_STACKED)) {
swayc_t *container = new_container(newview, parent_container->layout);
set_focused_container(newview);
arrange_windows(container, -1, -1);
}
} else { } else {
swayc_t *output = swayc_parent_by_type(focused, C_OUTPUT); swayc_t *output = swayc_parent_by_type(focused, C_OUTPUT);
wlc_handle *h = malloc(sizeof(wlc_handle)); wlc_handle *h = malloc(sizeof(wlc_handle));

View file

@ -66,6 +66,11 @@ void add_child(swayc_t *parent, swayc_t *child) {
if (!parent->focused) { if (!parent->focused) {
parent->focused = child; parent->focused = child;
} }
// wrap view into a container
if (parent->type == C_WORKSPACE && child->type == C_VIEW) {
new_container(child, parent->workspace_layout);
}
} }
void insert_child(swayc_t *parent, swayc_t *child, int index) { void insert_child(swayc_t *parent, swayc_t *child, int index) {
@ -80,6 +85,11 @@ void insert_child(swayc_t *parent, swayc_t *child, int index) {
if (!parent->focused) { if (!parent->focused) {
parent->focused = child; parent->focused = child;
} }
// wrap view into a container
if (parent->type == C_WORKSPACE && child->type == C_VIEW) {
new_container(child, parent->workspace_layout);
}
} }
void add_floating(swayc_t *ws, swayc_t *child) { void add_floating(swayc_t *ws, swayc_t *child) {
@ -302,7 +312,8 @@ void move_container(swayc_t *container, enum movement_direction dir) {
} }
// Change parent layout if we need to // Change parent layout if we need to
if (parent->children->length == 1 && parent->layout != layout) { if (parent->children->length == 1 && parent->layout != layout) {
parent->layout = layout; swayc_change_layout(parent, layout);
/* parent->layout = layout; */
continue; continue;
} }
if (parent->type == C_WORKSPACE) { if (parent->type == C_WORKSPACE) {