?
avatar BrainforgeUK
BrainforgeUK
25 Apr 2021

Steps to reproduce the issue

Add a 3rd part menu module to site.
Such as MenuCk
maximenuck-pro_9.1.11.zip
mobilemenuck-pro_1.3.3.zip

Expected result

When viewed on PC as window width is reduced except the mobile menu burger bar image to kick in.

Actual result

Something else kicks in and messed up what MenuCK is trying to do.
Turns out it its the bootstrap collapse code in the cassiopeia temlate (index.php and error.php).

System information (as much as possible)

Joomla 4 Beta 7
PHP 7.4

Additional comments

I suggestscreen shot 2021-04-25 at 08 25 48 a 'collpsiblenavbar' option be added to cassiopeia.

if ($this->params->get('collapsiblenavbar', 1)) :
    HTMLHelper::_('bootstrap.collapse', '.navbar-toggler');
    ?>
    <button class="navbar-toggler navbar-toggler-right" type="button" data-bs-toggle="collapse" data-bs-target="#navbar" aria-controls="navbar" aria-expanded="false" aria-label="<?php echo Text::_('TPL_BFUKCASSIOPEIA_TOGGLE'); ?>">
        <span class="icon-menu" aria-hidden="true"></span>
    </button>
    <div class="collapse navbar-collapse" id="navbar">
        <jdoc:include type="modules" name="menu" style="none" />
    </div>
    <?php
else :
    ?>
    <div id="navbar">
        <jdoc:include type="modules" name="menu" style="none" />
    </div>
    <?php
endif;

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
5.00

avatar BrainforgeUK BrainforgeUK - open - 25 Apr 2021
avatar joomla-cms-bot joomla-cms-bot - labeled - 25 Apr 2021
avatar BrainforgeUK
BrainforgeUK - comment - 25 Apr 2021

For those confused by the image location in above ... I wrote -

I suggestscreen a 'collpsiblenavbar' option be added to cassiopeia.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33293.

avatar dgrammatiko
dgrammatiko - comment - 25 Apr 2021

I sugges a 'collpsiblenavbar' option be added to cassiopeia.

The functionality doesn't even belong to the template, it's the menu' module responsibility. At the end of the day, templates should only compose the component of the current route and all the modules that are enabled for that route...

avatar BrainforgeUK BrainforgeUK - change - 28 Jul 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-07-28 11:22:19
Closed_By BrainforgeUK
avatar BrainforgeUK BrainforgeUK - close - 28 Jul 2021

Add a Comment

Login with GitHub to post a comment