laravel-menus Bootstrap 4

Hi, https://blackrockdigital.github.io/startbootstrap-modern-business/ As you can see in the Bootstrap 4 menu, the first lavel of link has the class "nav-link". For example

<a class="nav-link" href="about.html">About</a> In the second level the class is different, For example. <a class="dropdown-item" href="portfolio-3-col.html">3 Column Portfolio</a>

As I see in the presenter there is only one place to set de class

    public function getMenuWithoutDropdownWrapper($item)
    {
        return '<li class="nav-item" ' . $this->getActiveState($item) . '><a class="nav-link" href="' . $item->getUrl() . '" ' . $item->getAttributes() . '>' . $item->getIcon() . ' ' . $item->title . '</a></li>' . PHP_EOL;
    }

How can we set the correct class in function of depth?

Thanks and best regards

Sep 11 '17 15:09 drjoju

you can create a new presenter class for bootstrap 4.

Sep 11 '17 15:09 nWidart

Yes, I do it but I don't know how to process the second level. I created a presenter like this one but I don't know if it's the best solution `

' . PHP_EOL; } public function getCloseTagWrapper() { return PHP_EOL . '' . PHP_EOL; } public function getMenuWithoutDropdownWrapper($item) { return '

getActiveState($item) . '>getAttributes() . '>' . $item->getIcon() . ' ' . $item->title . '

' . PHP_EOL; } public function getSubmenuWithoutDropdownWrapper($item) { return '

getActiveState($item) . '>getAttributes() . '>' . $item->getIcon() . ' ' . $item->title . '

' . PHP_EOL; } public function getActiveState($item, $state = ' class="active"') { return $item->isActive() ? $state : null; } public function getActiveStateOnChild($item, $state = 'active') { return $item->hasActiveOnChild() ? $state : null; } public function getDividerWrapper() { return '

'; } public function getHeaderWrapper($item) { return '

'; } public function getMenuWithDropDownWrapper($item) { return '' . PHP_EOL; } public function getMultiLevelDropdownWrapper($item) { return '' . PHP_EOL; } public function getChildMenuItems(MenuItem $item) { $results = ''; foreach ($item->getChilds() as $child) { if ($child->hidden()) { continue; } if ($child->hasSubMenu()) { $results .= $this->getMultiLevelDropdownWrapper($child); } elseif ($child->isHeader()) { $results .= $this->getHeaderWrapper($child); } elseif ($child->isDivider()) { $results .= $this->getDividerWrapper(); } else { $results .= $this->getSubmenuWithoutDropdownWrapper($child); } } return $results; } } ` What do you think? Thanks

Sep 11 '17 15:09 drjoju

Plus format your post, this is unreadable :D

Sep 11 '17 15:09 nWidart

Sorry,

Basically I created a presenter based on NavbarPresenter and override the method getChildMenuItems.

The part modified is


            if ($child->hasSubMenu()) {
                $results .= $this->getMultiLevelDropdownWrapper($child);
            } elseif ($child->isHeader()) {
                $results .= $this->getHeaderWrapper($child);
            } elseif ($child->isDivider()) {
                $results .= $this->getDividerWrapper();
            } else {
                $results .= $this->getSubmenuWithoutDropdownWrapper($child);
            }

I introduce the method getSubmenuWithoutDropdownWrapper with this code


    public function getSubmenuWithoutDropdownWrapper($item)
    {
        return '<li class="nav-item" ' . $this->getActiveState($item) . '><a class="dropdown-item" href="' . $item->getUrl() . '" ' . $item->getAttributes() . '>' . $item->getIcon() . ' ' . $item->title . '</a></li>' . PHP_EOL;
    }

This method write the correct class (dropdown-item) in the submenu.

What do you think about this aproach?

Thanks

Sep 11 '17 18:09 drjoju

+1 for BS4 support out of the box. Would be great. Will see if I can submit a PR.