From 38b13b5bd996405c6a8fb72d2ececdaf1468231e Mon Sep 17 00:00:00 2001
From: John Mizerek <mizerek@gmail.com>
Date: Wed, 28 Jan 2026 16:39:23 -0800
Subject: [PATCH] Fix blank Manage Menus modal by setting title/body before
 showing

showMenuManager() was passing arguments to showModal() which accepts
none, so the modal content was silently discarded. Now follows the
same pattern as all other modal callers in the file.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 portal/menu-builder.html | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/portal/menu-builder.html b/portal/menu-builder.html
index 50f8d25..cdb465f 100644
--- a/portal/menu-builder.html
+++ b/portal/menu-builder.html
@@ -3420,9 +3420,9 @@
             </div>
           </div>
         `;
-        this.showModal('Manage Menus', content, [
-          { text: 'Close', primary: false, action: () => this.closeModal() }
-        ]);
+        document.getElementById('modalTitle').textContent = 'Manage Menus';
+        document.getElementById('modalBody').innerHTML = content;
+        this.showModal();
       },
 
       // Format days active bitmask to readable string