From e95bff7be04b8401c9d4e2b74fcb0117a4a6d1c7 Mon Sep 17 00:00:00 2001
From: Johan Malm <jgm323@gmail.com>
Date: Wed, 13 Nov 2024 21:11:35 +0000
Subject: [PATCH] menu: fix pipemenu bug causing parent menu to disappear

...introduced by 7651531632a345d65737e631c484b2e20d05a988

Only post-process the pipemenu instead of processing all menus because
post_processing() calls menu_update_scene() which is a bit heavier than
its predecessor and calls wlr_scene_node_set_enabled() which disables
existing menus.
---
 src/menu/menu.c | 23 +++++++++++++++++------
 1 file changed, 17 insertions(+), 6 deletions(-)

diff --git a/src/menu/menu.c b/src/menu/menu.c
index 966601eb..208a716a 100644
--- a/src/menu/menu.c
+++ b/src/menu/menu.c
@@ -367,6 +367,21 @@ menu_update_scene(struct menu *menu)
 	menu->size.height = item_y;
 }
 
+static void
+post_processing_from(struct server *server, struct menu *from)
+{
+	bool should_process = false;
+	struct menu *menu;
+	wl_list_for_each(menu, &server->menus, link) {
+		if (menu == from) {
+			should_process = true;
+		}
+		if (should_process) {
+			menu_update_scene(menu);
+		}
+	}
+}
+
 static void
 post_processing(struct server *server)
 {
@@ -1359,18 +1374,14 @@ create_pipe_menu(struct menu_pipe_context *ctx)
 	}
 	ctx->item->submenu = pipe_menu;
 
-	/*
-	 * TODO: refactor validate() and post_processing() to only
-	 * operate from current point onwards
-	 */
-
 	/* Set menu-widths before configuring */
-	post_processing(ctx->server);
+	post_processing_from(ctx->server, pipe_menu);
 
 	enum menu_align align = ctx->item->parent->align;
 	struct wlr_box pos = get_submenu_position(ctx->item, align);
 	menu_configure(pipe_menu, pos.x, pos.y, align);
 
+	/*TODO: refactor validate() to only operate from current point onwards */
 	validate(ctx->server);
 
 	/* Finally open the new submenu tree */