OliveStudio / olivemenus

A powerful menus plugin for Craft CMS built for the need of simplicity and flexibility
MIT License
40 stars 22 forks source link

menuHTML without div-container and outside UL #46

Closed niektenhoopen closed 4 years ago

niektenhoopen commented 4 years ago

I have added the functionality to request the menuHTML without the div-container and outside ul

Example:

<div>
    <ul>
        {{ craft.olivemenus.getMenuHTML('primary_menu', {
            'without-container': true,
            'without-ul': true,
        } ) }}
    </ul>
</div>

I use it to add extra list items to the UL that are variable based on the template

niektenhoopen commented 4 years ago

@alinistrate What do you think about this?

alinistrate commented 4 years ago

Niek. This is good. I will pull on Monday.

Thank you Alin

Get Outlook for Androidhttps://aka.ms/ghei36


From: Niek ten Hoopen notifications@github.com Sent: Saturday, May 16, 2020 11:01:57 AM To: OliveStudio/olivemenus olivemenus@noreply.github.com Cc: Alin Istrate alin.istrate@olivestudio.net; Mention mention@noreply.github.com Subject: Re: [OliveStudio/olivemenus] menuHTML without div-container and outside UL (#46)

@alinistratehttps://github.com/alinistrate What do you think about this?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/OliveStudio/olivemenus/pull/46#issuecomment-629606111, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABGOJUICR75WB6QDPVZMQ4DRRZB7LANCNFSM4NBSWWPA.

niektenhoopen commented 4 years ago

Niek. This is good. I will pull on Monday. Thank you Alin

Ok! I updated the date in the changelog.