From ea4ea8449966b8d51a48d3b11f88ac4ca402b05a Mon Sep 17 00:00:00 2001 From: tokyo4j Date: Mon, 10 Mar 2025 16:26:46 +0900 Subject: [PATCH] menu: use item->parent->id for client-list-combined-menu ...instead of item->id. --- src/menu/menu.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/menu/menu.c b/src/menu/menu.c index ebf16419..8139063c 100644 --- a/src/menu/menu.c +++ b/src/menu/menu.c @@ -1062,7 +1062,6 @@ update_client_list_combined_menu(struct server *server) buf_add(&buffer, title); current_item = item_create(menu, buffer.data, /*show arrow*/ false); - current_item->id = xstrdup(menu->id); current_item->client_list_view = view; fill_item("name.action", "Focus"); fill_item("name.action", "Raise"); @@ -1070,7 +1069,6 @@ update_client_list_combined_menu(struct server *server) } } current_item = item_create(menu, _("Go there..."), /*show arrow*/ false); - current_item->id = xstrdup(menu->id); fill_item("name.action", "GoToDesktop"); fill_item("to.action", workspace->name); } @@ -1682,7 +1680,7 @@ menu_execute_item(struct menuitem *item) * menu_close() and destroy_pipemenus() which we have to handle * before/after action_run() respectively. */ - if (item->id && !strcmp(item->id, "client-list-combined-menu") + if (!strcmp(item->parent->id, "client-list-combined-menu") && item->client_list_view) { actions_run(item->client_list_view, server, &item->actions, NULL); } else {