When viewing a group node, I see that the anchor for that group within the tree view is ...

<a href="/groups/admin-staff" class="">ADMIN STAFF</a>

I notice that the class has not been set.
Would it be sensible to de the class to 'current-node' or something like that - this way we can skin so as to highlight it.

Thank you

Comments

bschilt’s picture

bschilt commented

Yep, good catch. Would adding an 'active' class to the node that is being viewed then add and 'active-trail' to each of the ancestors be sufficient?

sirclickalot’s picture

sirclickalot
Primary language English
Location Somerset
commented

Indeed that would be a perfect solution.
Thank you.

bschilt’s picture

bschilt commented
Status: Active » Fixed

I have committed a fix for this issue. When viewing a subgroup node the menu link item in the tree will have the following class applied: og-subgroups-tree-active. Each parent element will then have this class applied: og-subgroups-tree-active-trail.

Also note that when ajax is enabled to load the initial tree, the markup changes a bit and the og-subgroups-tree-active-trail class is applied to the span tag instead of the li tag.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.