Add class "current" to all parents in a multi-level menu

[Claus]

Hello there!

I have a question regarding the "renderNav" function used in the ‘intermediate’ site-profile. I have a multi-level menu system going by using the renderNav function, and I’d like to extend the default "renderNav" function to also mark all the parents of any given page with "current".

I think I have to modify this line from the "renderNav" function:

$out .= $item->id == wire('page')->id ? "<li class='current'>" : "<li>";

… specifically adding something along the lines of foreach parent of the itemid "<li class='current'>" else "<li>", but is there a neat way of putting that into the existing line? Or how would you do it?

Thanks!

[diogo]

Not tested, but try this:

$page = wire('page');
$out .= $item->id == $page->id || $page->parents()->has($item) ? "<li class='current'>" : "<li>";

Edited November 16, 2014 by diogo
corrected the code

[Claus]

Hmm, that didn’t change anything as far as I can tell.

[diogo]

Sorry, that didn't make any sense I edited the code to have $page->parents()->has($item) instead of $page->parents()->has($page)

[Claus]

Awesomeness, it works! Thanks diogo!

[Soma]

Or simply use MarkupSimpleNavigation module.