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>
This commit is contained in:
2026-04-08 05:15:06 -05:00
parent 8d6a8df15e
commit 5a67111335

View File

@@ -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; ?>> <nav class="mod-menu mod-menu-main navbar navbar-expand-lg<?php echo $moduleclass_sfx; ?>"<?php echo $id; ?>>
<div class="container-fluid"> <div class="container-fluid">
<!-- Hamburger toggle button for mobile --> <!-- 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> <span class="fa-solid fa-bars" aria-hidden="true"></span>
</button> </button>
<!-- Collapsible menu content --> <!-- 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"> <ul class="navbar-nav mod-menu-main__list">
<?php foreach ($list as $i => &$item) : <?php foreach ($list as $i => &$item) :
$itemParams = $item->getParams(); $itemParams = $item->getParams();