Hi Alex

Thanks for answering my post.

I't works great with the (node.Level == 1) - the code is much simpler.

Do you have any idea how to solve the highlight part of the question? and how to make the menuitem with the dropdown "nonlinkable". So its only the children in the dropdown menu, that is clickable?

Thanks :)

Hi Alex

Thanks for answering, again :)

Your example worked perfectly when I switched the javascript:void(0) and @item.Url.

So the if statement looks like:

@if (itemHasChildren)
{
    <a href="javascript:void(0)">@item.Name</a>    
    @Traverse(item)
}
else
{
    <a href="@item.Url">@item.Name</a>
}

Why use javascript:void(0) instead of just # - Is it better practice?

Because if you press on link with # it will move your browser window to the top of the page, I don't like it

Of course you can use some js handler for it, but it's easier to use javascript:void(0)

Alex

Makes sense !

Thanks for the tip !

As I read it, I was supposed to switch the CurrentPage.Id == item.Id with item.Path.Contains(CurrentPage.Id)

The highlighting code throws an error:

The best overloaded method match for 'string.Contains(string)' has some invalid arguments

Guess it can't find the item.Path.Contains

Do you have any ideas what the problem is here?

Hi Anders

I rewrote your code to strongly typed, try it:

@inherits Umbraco.Web.Mvc.UmbracoViewPage

@{ var selection = Umbraco.AssignedContentItem.Site(); }

<nav id="nav">
    @Traverse(selection)
</nav>

@helper Traverse(IPublishedContent node)
{

var maxLevelForSitemap = 10;
var selection = node.Children.Where(x => x.IsVisible()).Where(x => x.Level <= maxLevelForSitemap);

if (selection.Any())
{
        <ul>
            @if (node.Level == 1)
            {
                <li class="@( Umbraco.AssignedContentItem.Url == "/" ? "current_page_item" : null)"><a href="/">Forside</a></li>
            }

            @foreach (var item in selection)
            {
                <li class="[email protected] @(item.Path.Contains(Umbraco.AssignedContentItem.Id.ToString()) ? "current_page_item" : null)">
                    @{
                        var itemHasChildren = item.Children.Where("Visible").Where("Level <= " + maxLevelForSitemap).Any();
                    }
                    @if (itemHasChildren)
                    {
                        <a href="javascript:void(0)">@item.Name</a>
                        @Traverse(item)
                    }
                    else
                    {
                        <a href="@item.Url">@item.Name</a>
                    }
                </li>
            }
        </ul>
    }
}

Hi Anders

Did you try latest code snippet?

Alex

Hi Alex

Sorry for not answering to your reply.

I've tried the code example you gave me, but it didn't work as intended.

I don't know if I asked the right question. But to make it clearer, I'm trying to highlight the menu item that has one of its "children" activated, and highlight the activated child.

Example:

        Menu Item1  -   Menu Item 2 - (((Menu Item 3)))
                                        Submenu item 1
                                        Submenu item 2
                                     (((Submenu item 3)))                              
                                        Submenu item 4

((( means highlighted )))

Hope the example helps, otherwise don't hesitate to ask :)

Hi Alex

The following error occurs:

Severity Code Description Project File Line Suppression State Error CS1973 'IPublishedContent' has no applicable method named 'IsDescendantOrSelf' but appears to have an extension method by that name. Extension methods cannot be dynamically dispatched. Consider casting the dynamic arguments or calling the extension method without the extension method syntax.
ViewsPartialsumbTopNavigation.cshtml

Do you know how to casting the dynamic arguments or call the extensions method without the extension method syntax?