Hello,
I was having a very minor navigation issue with this module. Attached to this issue is a proposed patch that will slightly modify the functionality of building the active trail. I appreciate your consideration in adding this to your module. Here's the issue:
When a node has multiple taxonomy terms in the same field, the menu trail was always defaulting to the first taxonomy term ID in the field.
This was a little confusing, especially when the user was coming to the node from a different taxonomy term (by clicking a link in the taxonomy term's page node list).
The breadcrumb would be built for a different taxonomy term than they clicked off of, so navigating back to the original taxonomy term isn't possible via the breadcrumb.
The attached patch just sets a $_SESSION variable when the user visits a taxonomy term page. Then when a user visits a node with a taxonomy term field set to have taxonomy menu trails, the code checks if the field has the taxonomy term ID in $_SESSION, and if so sets that term as the menu trail.
Thanks!
-Joe
Comment | File | Size | Author |
---|---|---|---|
taxonomy-menu-trail-context.patch | 1.45 KB | joecorall |