Bugfix: Correctly parse `move ... workspace *_on_output`
Patch status: merged
Patch by Steven Allen
Long description:
The parse spec for `move ... workspace ...` ordered next/prev before next_on_output/prev_on_output causing the parser to match next/prev before next_on_output/prev_on_output. Ticket: http://bugs.i3wm.org/report/ticket/941
To apply this patch, use:
curl http://cr.i3wm.org/patch/32/raw.patch | git am
b/parser-specs/commands.spec
| 18 |
@@ -223,7 +223,7 @@ state RENAME_WORKSPACE_NEW_NAME: |
| 19 |
-> call cmd_rename_workspace($old_name, $new_name) |
| 20 |
|
| 21 |
# move <direction> [<pixels> [px]] |
| 22 |
-# move [window|container] [to] workspace [<str>|next|prev|current] |
| 23 |
+# move [window|container] [to] workspace [<str>|next|prev|next_on_output|prev_on_output|current] |
| 24 |
# move [window|container] [to] output <str> |
| 25 |
# move [window|container] [to] scratchpad |
| 26 |
# move workspace to [output] <str> |
| 27 |
@@ -264,7 +264,7 @@ state MOVE_DIRECTION_PX: |
| 28 |
state MOVE_WORKSPACE: |
| 29 |
'to' |
| 30 |
-> MOVE_WORKSPACE_TO_OUTPUT |
| 31 |
- workspace = 'next', 'prev', 'next_on_output', 'prev_on_output', 'current' |
| 32 |
+ workspace = 'next_on_output', 'prev_on_output', 'next', 'prev', 'current' |
| 33 |
-> call cmd_move_con_to_workspace($workspace) |
| 34 |
'back_and_forth' |
| 35 |
-> call cmd_move_con_to_workspace_back_and_forth() |