Hi Lars,

I think that you can do it like this.

@{var selection = Model.Content.Site().Children.Where(x => x.IsVisible());}

<ul>
    @foreach(var item in selection)
    {
        <li class="@(item.IsAncestorOrSelf(Model.Content) ? "current" : null)">
            <a href="@item.Url">@item.Name</a>
        </li>
    }
</ul>

Hope this helps,

/Dennis

Hello,

You could also build your menu in a controller and render that. That way it's strongly typed and no linq queries in your views. For example:

Menu logic
https://github.com/jbreuer/Hybrid-Framework-for-Umbraco-v7-Best-Practises/blob/master/Umbraco.Extensions/BLL/ModelLogic.cs#L44

Menu rendering:
https://github.com/jbreuer/Hybrid-Framework-for-Umbraco-v7-Best-Practises/blob/master/Umbraco.Site/Views/Master.cshtml#L32 

Have a look at the Hybrid Framework.

Jeroen

I was doing it like Dennis' second example but then when I set up child doctypes on pages with the menu it started throwing an error.  I couldn't figure out why but using the "Visible" dynamic query feature instead of linq fixed it for some reason.  May have been some localised issue to my code but if you get it try doing like this:

var selection =Model.Content.AncestorOrSelf(1).Children.Where("Visible");

Hi Both

This was the solution. Now it is strongly typed. Thank you for your help :-)

/Lars

@{  IPublishedContent root = Model.Content.AncestorOrSelf(1); }

<ul>
    <li class="@((Model.Content.Name == root.Name) ? "active" : null)"><a href="/">@root.Name</a></li>

    @foreach (IPublishedContent menuItem in root.Children.Where(x => x.IsVisible()))
    {
            else
            {
                <li class="@((menuItem.Name == Model.Content.Name) ? "active" : null)">
                    <a href="@menuItem.Url">@menuItem.Name</a>
                </li>
            }
        }
}

</ul>

is working on a reply...