diff --git a/include/ewmh.h b/include/ewmh.h index 01ae67f96..f616eb840 100644 --- a/include/ewmh.h +++ b/include/ewmh.h @@ -11,6 +11,12 @@ #include +/** + * Updates all the EWMH desktop properties. + * + */ +void ewmh_update_desktop_properties(void); + /** * Updates _NET_CURRENT_DESKTOP with the current desktop number. * @@ -20,24 +26,6 @@ */ void ewmh_update_current_desktop(void); -/** - * Updates _NET_NUMBER_OF_DESKTOPS which we interpret as the number of - * noninternal workspaces. - */ -void ewmh_update_number_of_desktops(void); - -/** - * Updates _NET_DESKTOP_NAMES: "The names of all virtual desktops. This is a - * list of NULL-terminated strings in UTF-8 encoding" - */ -void ewmh_update_desktop_names(void); - -/** - * Updates _NET_DESKTOP_VIEWPORT, which is an array of pairs of cardinals that - * define the top left corner of each desktop's viewport. - */ -void ewmh_update_desktop_viewport(void); - /** * Updates _NET_WM_DESKTOP for all windows. * A request will only be made if the cached value differs from the calculated value. diff --git a/src/commands.c b/src/commands.c index 624c27dbd..db0b075d9 100644 --- a/src/commands.c +++ b/src/commands.c @@ -2002,9 +2002,7 @@ void cmd_rename_workspace(I3_CMD, const char *old_name, const char *new_name) { cmd_output->needs_tree_render = true; ysuccess(true); - ewmh_update_desktop_names(); - ewmh_update_desktop_viewport(); - ewmh_update_current_desktop(); + ewmh_update_desktop_properties(); startup_sequence_rename_workspace(old_name_copy, new_name); free(old_name_copy); diff --git a/src/ewmh.c b/src/ewmh.c index a26eef39c..7bd23fb78 100644 --- a/src/ewmh.c +++ b/src/ewmh.c @@ -39,7 +39,7 @@ void ewmh_update_current_desktop(void) { * Updates _NET_NUMBER_OF_DESKTOPS which we interpret as the number of * noninternal workspaces. */ -void ewmh_update_number_of_desktops(void) { +static void ewmh_update_number_of_desktops(void) { Con *output, *ws; static uint32_t old_idx = 0; uint32_t idx = 0; @@ -61,7 +61,7 @@ void ewmh_update_number_of_desktops(void) { * Updates _NET_DESKTOP_NAMES: "The names of all virtual desktops. This is a * list of NULL-terminated strings in UTF-8 encoding" */ -void ewmh_update_desktop_names(void) { +static void ewmh_update_desktop_names(void) { Con *output, *ws; int msg_length = 0; @@ -88,7 +88,7 @@ void ewmh_update_desktop_names(void) { * Updates _NET_DESKTOP_VIEWPORT, which is an array of pairs of cardinals that * define the top left corner of each desktop's viewport. */ -void ewmh_update_desktop_viewport(void) { +static void ewmh_update_desktop_viewport(void) { Con *output, *ws; int num_desktops = 0; /* count number of desktops */ @@ -109,6 +109,18 @@ void ewmh_update_desktop_viewport(void) { A__NET_DESKTOP_VIEWPORT, XCB_ATOM_CARDINAL, 32, current_position, &viewports); } +/* + * Updates all the EWMH desktop properties. + * + */ +void ewmh_update_desktop_properties(void) { + ewmh_update_number_of_desktops(); + ewmh_update_desktop_viewport(); + ewmh_update_current_desktop(); + ewmh_update_desktop_names(); + ewmh_update_wm_desktop(); +} + static void ewmh_update_wm_desktop_recursively(Con *con, const uint32_t desktop) { Con *child; diff --git a/src/main.c b/src/main.c index 3ebdbf0cc..9f69834fd 100644 --- a/src/main.c +++ b/src/main.c @@ -852,10 +852,7 @@ int main(int argc, char *argv[]) { ewmh_update_workarea(); /* Set the ewmh desktop properties. */ - ewmh_update_current_desktop(); - ewmh_update_number_of_desktops(); - ewmh_update_desktop_names(); - ewmh_update_desktop_viewport(); + ewmh_update_desktop_properties(); struct ev_io *xcb_watcher = scalloc(1, sizeof(struct ev_io)); xcb_prepare = scalloc(1, sizeof(struct ev_prepare)); diff --git a/src/tree.c b/src/tree.c index f93b2262e..4057d177b 100644 --- a/src/tree.c +++ b/src/tree.c @@ -301,9 +301,7 @@ bool tree_close_internal(Con *con, kill_window_t kill_window, bool dont_kill_par if (ws == con) { DLOG("Closing workspace container %s, updating EWMH atoms\n", ws->name); - ewmh_update_number_of_desktops(); - ewmh_update_desktop_names(); - ewmh_update_wm_desktop(); + ewmh_update_desktop_properties(); } con_free(con); diff --git a/src/workspace.c b/src/workspace.c index 3cf747542..597057985 100644 --- a/src/workspace.c +++ b/src/workspace.c @@ -161,10 +161,7 @@ Con *workspace_get(const char *num, bool *created) { con_attach(workspace, content, false); ipc_send_workspace_event("init", workspace, NULL); - ewmh_update_number_of_desktops(); - ewmh_update_desktop_names(); - ewmh_update_desktop_viewport(); - ewmh_update_wm_desktop(); + ewmh_update_desktop_properties(); if (created != NULL) *created = true; } else if (created != NULL) { @@ -520,10 +517,7 @@ void workspace_show(Con *workspace) { old_focus = NULL; } - ewmh_update_number_of_desktops(); - ewmh_update_desktop_names(); - ewmh_update_desktop_viewport(); - ewmh_update_wm_desktop(); + ewmh_update_desktop_properties(); } } diff --git a/testcases/t/294-update-ewmh-atoms.t b/testcases/t/294-update-ewmh-atoms.t index 047cc1197..f13b17645 100644 --- a/testcases/t/294-update-ewmh-atoms.t +++ b/testcases/t/294-update-ewmh-atoms.t @@ -101,11 +101,19 @@ is_deeply(\@actual_names, \@expected_names); # Kill first window to close a workspace. cmd '[id="' . $second->id . '"] kill'; -is(get_current_desktop, 2, '_NET_CURRENT_DESKTOP should be updated'); +is(get_current_desktop, 1, '_NET_CURRENT_DESKTOP should be updated'); is(get_num_of_desktops, 2, '_NET_NUMBER_OF_DESKTOPS should be updated'); my @actual_names = get_desktop_names; my @expected_names = ('0', '2'); is_deeply(\@actual_names, \@expected_names, '_NET_DESKTOP_NAMES should be updated'); +# Rename workspace to reorder them. +cmd 'rename workspace 0 to 5'; + +is(get_current_desktop, 0, '_NET_CURRENT_DESKTOP should be updated'); +is(get_num_of_desktops, 2, '_NET_NUMBER_OF_DESKTOPS should remain the same'); +my @actual_names = get_desktop_names; +my @expected_names = ('2', '5'); +is_deeply(\@actual_names, \@expected_names, '_NET_DESKTOP_NAMES should be updated'); done_testing;