diff options
| author | Ronan Pigott <rpigott@berkeley.edu> | 2020-01-14 17:50:55 -0700 |
|---|---|---|
| committer | Brian Ashworth <bosrsf04@gmail.com> | 2020-01-16 21:48:25 -0500 |
| commit | 72ad35ba16b30960fa49f25fac165a54009bcd03 (patch) | |
| tree | 2251cc1d2af7cc648974b3a93095d2e9ccc9d8ec /sway/commands | |
| parent | d88d4bbd9d363435a4d65191446a497de9031cca (diff) | |
cmd_move: Fix crash when moving to scratchpad hidden split containers
Diffstat (limited to 'sway/commands')
| -rw-r--r-- | sway/commands/move.c | 21 |
1 files changed, 13 insertions, 8 deletions
diff --git a/sway/commands/move.c b/sway/commands/move.c index 6ad1c7ca..09a47652 100644 --- a/sway/commands/move.c +++ b/sway/commands/move.c @@ -248,10 +248,11 @@ static void container_move_to_container(struct sway_container *container, ipc_event_window(container, "move"); } - workspace_focus_fullscreen(destination->workspace); + if (destination->workspace) { + workspace_focus_fullscreen(destination->workspace); + workspace_detect_urgent(destination->workspace); + } - // Update workspace urgent state - workspace_detect_urgent(destination->workspace); if (old_workspace && old_workspace != destination->workspace) { workspace_detect_urgent(old_workspace); } @@ -519,8 +520,10 @@ static struct cmd_results *cmd_move_container(bool no_auto_back_and_forth, } struct sway_output *new_output = node_get_output(destination); - struct sway_workspace *new_output_last_ws = old_output == new_output ? - NULL : output_get_active_workspace(new_output); + struct sway_workspace *new_output_last_ws = NULL; + if (new_output && old_output != new_output) { + new_output_last_ws = output_get_active_workspace(new_output); + } // save focus, in case it needs to be restored struct sway_node *focus = seat_get_focus(seat); @@ -551,12 +554,14 @@ static struct cmd_results *cmd_move_container(bool no_auto_back_and_forth, } // restore focus on destination output back to its last active workspace - struct sway_workspace *new_workspace = - output_get_active_workspace(new_output); - if (!sway_assert(new_workspace, "Expected output to have a workspace")) { + struct sway_workspace *new_workspace = new_output ? + output_get_active_workspace(new_output) : NULL; + if (new_output && + !sway_assert(new_workspace, "Expected output to have a workspace")) { return cmd_results_new(CMD_FAILURE, "Expected output to have a workspace"); } + if (new_output_last_ws && new_output_last_ws != new_workspace) { struct sway_node *new_output_last_focus = seat_get_focus_inactive(seat, &new_output_last_ws->node); |