Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#13865 closed defect (fixed)

[PATCH] ProjectMenuPlugin prevents active navigation item from being highlighted

Reported by: anonymous Owned by: Ryan J Ollos
Priority: normal Component: ProjectMenuPlugin
Severity: normal Keywords:
Cc: Trac Release: 1.4

Description

I was trying out this plugin on my new Trac 1.4.2 server build, and found that it prevented the active/current navigation item from being highlighted.

It seems that generally this kind of behavior can happen at odd times (when there are errors?), but there did not appear to be any errors shown on the web pages or the log.

I am attaching a patch that fixes the issue, which involves moving the add_script() call and changing the parameters and return value of post_process_request().

I don't know why this fixes the problem. Obviously there's something in the core that doesn't like the previous behavior (even though generally things were working).

I'm not sure if I will end up using this plugin (though it could be convenient) because I think it will be too slow, although I think it's only a hit when the project is first accessed. Right now I'm just testing with a subset of projects.

Attachments (1)

projectmenuplugin_fix_active_nav_trac_142.patch (882 bytes) - added by anonymous 4 years ago.
Fix to allow active navigation item to be highlighted.

Download all attachments as: .zip

Change History (5)

Changed 4 years ago by anonymous

Fix to allow active navigation item to be highlighted.

comment:1 Changed 4 years ago by anonymous

P.S. I just realized, it would probably be prudent to eliminate the reference to Genshi.

from trac.util.html import html as tag

comment:2 Changed 4 years ago by Ryan J Ollos

Owner: set to Ryan J Ollos
Status: newaccepted

comment:3 Changed 4 years ago by Ryan J Ollos

Resolution: fixed
Status: acceptedclosed

In 17833:

TracProjectMenu 2.1dev: Make compatible with Trac >= 1.2

Fixes #13865.

comment:4 Changed 4 years ago by Ryan J Ollos

Changes are untested. Please let me know if you encounter any problems.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.