aboutsummaryrefslogtreecommitdiff
path: root/sway
diff options
context:
space:
mode:
Diffstat (limited to 'sway')
-rw-r--r--sway/commands/workspace.c66
-rw-r--r--sway/sway.5.txt7
2 files changed, 43 insertions, 30 deletions
diff --git a/sway/commands/workspace.c b/sway/commands/workspace.c
index 14fe242f..7db1f332 100644
--- a/sway/commands/workspace.c
+++ b/sway/commands/workspace.c
@@ -12,12 +12,44 @@ struct cmd_results *cmd_workspace(int argc, char **argv) {
if ((error = checkarg(argc, "workspace", EXPECTED_AT_LEAST, 1))) {
return error;
}
- if (argc == 1 || (argc >= 2 && strcasecmp(argv[0], "number") == 0) ) {
+
+ int output_location = -1;
+
+ for (int i = 0; i < argc; ++i) {
+ if (strcasecmp(argv[i], "output") == 0) {
+ output_location = i;
+ break;
+ }
+ }
+ if (output_location >= 0) {
+ if ((error = checkarg(argc, "workspace", EXPECTED_EQUAL_TO, output_location + 2))) {
+ return error;
+ }
+ struct workspace_output *wso = calloc(1, sizeof(struct workspace_output));
+ if (!wso) {
+ return cmd_results_new(CMD_FAILURE, "workspace output",
+ "Unable to allocate workspace output");
+ }
+ wso->workspace = join_args(argv, argc - 2);
+ wso->output = strdup(argv[output_location]);
+ int i = -1;
+ if ((i = list_seq_find(config->workspace_outputs, workspace_output_cmp_workspace, wso)) != -1) {
+ struct workspace_output *old = config->workspace_outputs->items[i];
+ free(old); // workspaces can only be assigned to a single output
+ list_del(config->workspace_outputs, i);
+ }
+ sway_log(L_DEBUG, "Assigning workspace %s to output %s", wso->workspace, wso->output);
+ list_add(config->workspace_outputs, wso);
+ if (!config->reading) {
+ // TODO: Move workspace to output. (don't do so when reloading)
+ }
+ }
+ else {
if (config->reading || !config->active) {
return cmd_results_new(CMD_DEFER, "workspace", NULL);
}
swayc_t *ws = NULL;
- if (argc >= 2) {
+ if (strcasecmp(argv[0], "number") == 0) {
if (!(ws = workspace_by_number(argv[1]))) {
char *name = join_args(argv + 1, argc - 1);
ws = workspace_create(name);
@@ -41,9 +73,11 @@ struct cmd_results *cmd_workspace(int argc, char **argv) {
ws = workspace_create(prev_workspace_name);
}
} else {
- if (!(ws = workspace_by_name(argv[0]))) {
- ws = workspace_create(argv[0]);
+ char *name = join_args(argv, argc);
+ if (!(ws = workspace_by_name(name))) {
+ ws = workspace_create(name);
}
+ free(name);
}
swayc_t *old_output = swayc_active_output();
workspace_switch(ws);
@@ -55,30 +89,6 @@ struct cmd_results *cmd_workspace(int argc, char **argv) {
center_pointer_on(focused);
}
}
- } else {
- if (strcasecmp(argv[1], "output") == 0) {
- if ((error = checkarg(argc, "workspace", EXPECTED_EQUAL_TO, 3))) {
- return error;
- }
- struct workspace_output *wso = calloc(1, sizeof(struct workspace_output));
- if (!wso) {
- return cmd_results_new(CMD_FAILURE, "workspace output",
- "Unable to allocate workspace output");
- }
- wso->workspace = strdup(argv[0]);
- wso->output = strdup(argv[2]);
- int i = -1;
- if ((i = list_seq_find(config->workspace_outputs, workspace_output_cmp_workspace, wso)) != -1) {
- struct workspace_output *old = config->workspace_outputs->items[i];
- free(old); // workspaces can only be assigned to a single output
- list_del(config->workspace_outputs, i);
- }
- sway_log(L_DEBUG, "Assigning workspace %s to output %s", argv[0], argv[2]);
- list_add(config->workspace_outputs, wso);
- if (!config->reading) {
- // TODO: Move workspace to output. (don't do so when reloading)
- }
- }
}
return cmd_results_new(CMD_SUCCESS, NULL, NULL);
}
diff --git a/sway/sway.5.txt b/sway/sway.5.txt
index f425cfe7..8e3f64ab 100644
--- a/sway/sway.5.txt
+++ b/sway/sway.5.txt
@@ -364,8 +364,11 @@ The default colors are:
be configured with perfectly aligned adjacent positions for this option to
have any effect.
-**workspace** <name>::
- Switches to the specified workspace.
+**workspace** [number] <name>::
+ Switches to the specified workspace. The string "number" is optional. The
+ worspace _name_, if unquoted, may not contain the string "output", as sway
+ will assume that the command is moving a worspace to an output, as described
+ below.
**workspace** <prev|next>::
Switches to the next workspace on the current output or on the next output