Merge pull request #3327 from orestisf1993/rename-mouse-bug
cmd_rename_workspace: always call con_focus
This commit is contained in:
commit
dbf4e1b95d
|
@ -2021,11 +2021,13 @@ void cmd_rename_workspace(I3_CMD, const char *old_name, const char *new_name) {
|
|||
}
|
||||
workspace_move_to_output(workspace, target_output);
|
||||
|
||||
break;
|
||||
}
|
||||
|
||||
bool can_restore_focus = previously_focused != NULL;
|
||||
/* NB: If previously_focused is a workspace we can't
|
||||
* work directly with it since it might have been cleaned up by
|
||||
* workspace_show() already, depending on the
|
||||
* focus order/number of other workspaces on the output.
|
||||
/* NB: If previously_focused is a workspace we can't work directly with it
|
||||
* since it might have been cleaned up by workspace_show() already,
|
||||
* depending on the focus order/number of other workspaces on the output.
|
||||
* Instead, we loop through the available workspaces and only focus
|
||||
* previously_focused if we still find it. */
|
||||
if (previously_focused_content) {
|
||||
|
@ -2037,10 +2039,7 @@ void cmd_rename_workspace(I3_CMD, const char *old_name, const char *new_name) {
|
|||
if (can_restore_focus) {
|
||||
/* Restore the previous focus since con_attach messes with the focus. */
|
||||
workspace_show(con_get_workspace(previously_focused));
|
||||
con_activate(previously_focused);
|
||||
}
|
||||
|
||||
break;
|
||||
con_focus(previously_focused);
|
||||
}
|
||||
|
||||
cmd_output->needs_tree_render = true;
|
||||
|
|
Loading…
Reference in New Issue