if submenu not same as default, the top-menu to rendered

[nathandunn]

If I have two menus:

[A] B C
[1] 2 3

and [A] is selected with default [1]

If I select [2] it should be:

[A] B C
1 [2] 3

But instead [A] is not highlighted.

Using this type of notation:
<nav:render group="main-menu" />
<nav:renderSubItems group="main-menu"/>

(If I used <nav:render group="main-menu" submenu="true"/> it shows all of them . . . .not ideal)

To fix in NavigationTagLibrary.groovy I added an index:

// in eachItem (subItem is probably correct, but probably doesn't handle multiple levels):

        iterateItems(items, var, body, attrs.params, activePath, 0)


protected iterateItems = { items, var, body, linkParams, activePath, index = -1 ->

. . .

       if (index>=0) {
                data.active = pathIsActive(item.path[index], activePath[index])
            }
            else{
                data.active = pathIsActive(item.path, activePath)
            }

[nathandunn]

Still pretty hacky on my part . . . needed to account for instances without subItems where top-level points to different places within the same controller . .. Looking forward to your rewrite / more elegant solution:

if (index >= 0 && item.subItems) {
data.active = pathIsActive(item.path[index], activePath[index])
}
else {
data.active = pathIsActive(item.path, activePath)
}

[nathandunn]

I take that back . . you definitely DO NOT want to have item.subItems in the if statement.

What you need is a guarantee that if more than one top-level argument is selected, it only chooses one.

i.e. if A=price.edit and B=price

B should be highlighted in every case except price.edit . . in which case A should be highlighted (you should never have more than one highlighted / active).

if (index >= 0) {
data.active = pathIsActive(item.path[index], activePath[index])
}
else {
data.active = pathIsActive(item.path, activePath)
}