From 5a67111335b714f6488543818f1fca33d625981b Mon Sep 17 00:00:00 2001
From: Jonathan Miller <230051081+jmiller-moko@users.noreply.github.com>
Date: Wed, 8 Apr 2026 05:15:06 -0500
Subject: [PATCH] Use unique collapse ID for mainmenu too

Append module ID to collapse target so multiple mainmenu
instances don't conflict.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 src/html/mod_menu/mainmenu.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/html/mod_menu/mainmenu.php b/src/html/mod_menu/mainmenu.php
index e664177..1ee4c96 100644
--- a/src/html/mod_menu/mainmenu.php
+++ b/src/html/mod_menu/mainmenu.php
@@ -30,12 +30,12 @@ $moduleclass_sfx = htmlspecialchars($params->get('moduleclass_sfx', ''), ENT_COM
 <nav class="mod-menu mod-menu-main navbar navbar-expand-lg<?php echo $moduleclass_sfx; ?>"<?php echo $id; ?>>
     <div class="container-fluid">
         <!-- Hamburger toggle button for mobile -->
-        <button class="navbar-toggler" type="button" data-bs-toggle="collapse" data-bs-target="#mainMenuCollapse" aria-controls="mainMenuCollapse" aria-expanded="false" aria-label="Toggle Main Menu">
+        <button class="navbar-toggler" type="button" data-bs-toggle="collapse" data-bs-target="#mainMenuCollapse-<?php echo $module->id; ?>" aria-controls="mainMenuCollapse-<?php echo $module->id; ?>" aria-expanded="false" aria-label="Toggle Main Menu">
             <span class="fa-solid fa-bars" aria-hidden="true"></span>
         </button>
 
         <!-- Collapsible menu content -->
-        <div class="collapse navbar-collapse" id="mainMenuCollapse">
+        <div class="collapse navbar-collapse" id="mainMenuCollapse-<?php echo $module->id; ?>">
             <ul class="navbar-nav mod-menu-main__list">
 <?php foreach ($list as $i => &$item) :
     $itemParams = $item->getParams();