MokoConsulting/MokoCassiopeia
1
1
Fork 0
Code Issues 2 Pull Requests Actions Packages Releases 1 Activity

Fix horizontal menu: use own sub-layouts, unique collapse ID

Browse Source 
horizontal.php was referencing mainmenu_ sub-layouts instead of
horizontal_ ones. Also use module ID in collapse target to avoid
conflicts when multiple menus exist on the same page.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Jonathan Miller
2026-04-08 05:14:05 -05:00
parent 35b1c55d6f
commit 8d6a8df15e
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/html/mod_menu/horizontal.php
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="#horizontalMenuCollapse-<?php echo $module->id; ?>" aria-controls="horizontalMenuCollapse-<?php echo $module->id; ?>" aria-expanded="false" aria-label="Toggle 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="horizontalMenuCollapse-<?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();
@@ -80,11 +80,11 @@ $moduleclass_sfx = htmlspecialchars($params->get('moduleclass_sfx', ''), ENT_COM
case 'component': case 'component':
case 'heading': case 'heading':
case 'url': case 'url':
require ModuleHelper::getLayoutPath('mod_menu', 'mainmenu_' . $item->type); require ModuleHelper::getLayoutPath('mod_menu', 'horizontal_' . $item->type);
break; break;
default: default:
require ModuleHelper::getLayoutPath('mod_menu', 'mainmenu_url'); require ModuleHelper::getLayoutPath('mod_menu', 'horizontal_url');
break; break;
endswitch; endswitch;