Menu links whose path is <front> do not get the .active class assigned when the front page is displayed. Patch attached.

CommentFileSizeAuthor
l-patch.txt822 bytesbjaspan

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Needs work

I agree with the issue, but this patch doesn't solve the problem (or perhaps doesn't solve it any more?)

The workaround I found was that instead of using in my paths I use the same value that I have in my settings as the front page.

So, in settings front page is set to node/4
In menu, the link to "Home" is set to node/4

This seems to work in at least some of the cases.

coreb’s picture

coreb commented
Version: x.y.z » 5.x-dev

moving from x.y.z to a real version queue. My guess by the date is that it would be 5.x-dev. Please correct it if I'm wrong.

kickn’s picture

kickn commented

is there any update about this yet? is it in the cvs?

rkerr’s picture

rkerr commented

I think I've patched this before on a 4.7 site for a client.. now if I can remember what I did then I'll post my version of the patch. :)

Not sure why the current patch doesn't work, except maybe because of the triple-equals?

RobRoy’s picture

RobRoy commented
Status: Needs work » Closed (duplicate)

Duplicate of http://drupal.org/node/78487.

chromeyellow’s picture

chromeyellow commented

New patch at the issue that this duplicates (http://drupal.org/node/78487). Similar to this patch but instead of relying on drupal_is_front_page it tests against empty request + :

4.7.5 common.inc line 1119 changes to:

  if ($path == $_GET['q'] || ($path == '<front>' && !isset($_REQUEST['q'])) ) {

If interested, test and follow up at 78487...