Main navigation menu item is not highlighted when active

[anonymous]

The navigation menu items created by NavAddPlugin are not highlighted when they are active. For example, I added a "Open Ticket" menu item to the main navigation menu. After I click on the "Open Ticket" link in the main navigation, it is not highlighted. I expected it to be highlighted like for "Wiki" or "New Ticket" links.

I'm not sure how feasible this is to do. BTW, thanks for the great plugin!

[sramij@…]

I agree, i faced the same problem exactly. how can i fix it, any solutions?

Thanks

[sramij@…]

is there a patch which fix it? any suggestions?

[hihi]

I have the same problem :( , It would be great to have that feature !

[Michael Renzmann]

The buttons added by the plugin just link to other resources. Now imagine you add a button that links to /wiki/SomePage - how should the plugin (or Trac as a whole) know whether that page is visited by klicking on the button added by TracAddNav instead of one of the other ways to reach that page?

I'm open for suggestions, but I currently don't see a way to do that.

[jashugan@…]

Now imagine you add a button that links to /wiki/SomePage - how should the plugin (or Trac as a whole) know whether that page is visited by klicking on the button added by TracAddNav? instead of one of the other ways to reach that page?

It doesn't matter to me how I reached that page. If it is highlighted when I'm on the page, then that will be sufficient. I am willing to test it out!

[Flignet]

Why not add simply check if the xxx.title is equal to the current wiki page, if so highlight the button?

[Michael Renzmann]

Replying to Flignet:

Why not add simply check if the xxx.title is equal to the current wiki page, if so highlight the button?

Because this will lead to false results when the button links, for example, to a wiki page. You will have two highlighted buttons then, the regular Wiki one, and the one added with NavAddPlugin linking to this page. In my eyes this would be quite misleading, thus I'm not confident with this solution.

[NicolasDumoulin]

A solution could be to give the ability to put in the "url" an traclinks rather than an url. I think it sould be possible to hook the active page with this method … maybe.

[anonymous]

No solution to highlight the button when active?

[Ryan J Ollos]

See ticket #4914 for the MenusPlugin for a similar issue.

[Adam Bellaire]

Just an observation to the folks asking for this, the maintainer may be (is probably) already aware:

The way the plugin is implemented right now, you simply cannot indicate that you are on an active page. That's because the method get_active_navigation_item isn't invoked unless your INavigationProvider is *also* the IRequestHandler which is processing the current request and rendering the menu. This plugin just generates links, and doesn't actually implement the IRequestHandler interface. That wouldn't work anyway because the plugin would have to actually handle requests for the items it generates, and you want the actual handlers (e.g. the Wiki module) to do their work. Basically, doing this from within the python side would be almost impossible.

However, another approach could be taken: Implement this purely client-side using jQuery. Scan the URLs in the navigation menus, if one of them matches the menu location, add the .active class to that navigation item, and remove it from wherever it is currently. All you'd have to do is make your plugin an ITemplateProvider and provide the JS.

Here's a simple jQuery statement that I've tested which accomplishes this (for items in mainnav):

$('#mainnav ul li a').each(function() {
    if ($(this).attr('href') == location.pathname) {
           $('#mainnav ul li.active').removeClass('active');
           $(this).addClass('active');
    } 
});

[Michael Renzmann]

Replying to bellaire@ufl.edu:

All you'd have to do is make your plugin an ITemplateProvider and provide the JS.

Nice idea. I unfortunately don't have the time to implement that now, but if someone provides a patch I'd commit that.

[Adam Bellaire]

Patch to use client-side jQuery to highlight the active link

[Adam Bellaire]

Note there was a bug in my pasted code on 2/16 which didn't work. That's fixed in the patch I just uploaded.

[Mitar]

I have enabled that in my ​fork with server-side solution.

[Ryan J Ollos]

#8591 closed as a duplicate.

[walty8@…]

Replying to bellaire:

Fixed two minor issues in the code:

$(document).ready(function(){
    $('#mainnav ul li a').each(function() {
        var currentPath = location.pathname + location.search;
        if ($(this).attr('href') == currentPath) {
               $('#mainnav ul li.active').removeClass('active');
               $(this).parent().addClass('active');
        }
    });
});

[Ryan J Ollos]

I'm not sure that a client side solution is ideal, but I expect this plugin will soon be deprecated, so better to have a fix for the issue now.

[Ryan J Ollos]

Replying to mitar:

I have enabled that in my ​fork with server-side solution.

The repository appears to be deleted. Do you have an updated link?

[Ryan J Ollos]

In 15977:

0.4: Highlight active navigation item

Patch by bellair, revised by walty8.

Fixes #592.