1. Active menu items shouldn't be links. They should be unclickable text.
It's obvious that link should link somewhere else, but not to the same page, where user is now. There is no purpose in link, that links to itself.
2. active_trail is missing somewhere
Somewhere class="active_trail" does appear, somewhere it doesn't. But it should appear in every single menu item, if only somehow current node belongs to menu item.
Comments
Comment #1
yoroy commentedCritical is for stuff that's broken. This is not broken. I'm also quite certain there are open issues for both issues you mention. Please search before you post.
And open an issue for each point you are making and give examples, "active_trail is missing somewhere" isn't really helpful.
In the issue queue you have to be as specific and precise as possible to have any chance of people responding. :)
Comment #2
Mavros Gatos commentedOk, thanks for information.
Talking about active menu items, they have been links for years, so it's still important.
Comment #3
dpearcefl commentedClosing this issue because of a lack of activity.