active item: impossible to style differently for 2 menus

[teapotbag]

I have two menus, styled very differently.

I need to make the current active menu item stand out, but obviously, I have to apply different css styles to the active items in the two different menus.

I was able to do this in 1.5, but it seems impossible in 1.6, because the menu structure has changed.

It looks like this:

<ul class="menu-suffix">
<li id="item-25" class="current active">
<a href="link">ITEM</a>
</li>

The suffix is only added to the whole menu, but no longer to individual item classes. So the classes "current" and "active" are given to an active menu item, regardless which menu it belongs to. There is no way to style them separately for each menu!?

Any solutions for this?

I'm thinking it should look like this:
<li id="item-25" class="current-suffix active-suffix">

and I believe this is defined in modules/mod_menu/tmpl/default.php, but I don't know how to tweak that php myself?

If anyone can help I would be sooo grateful!

[teapotbag]

OH wait! I've figured it out! (copied the php from the 1.5 module file and woohooo it works!)

If anyone needs this in the future, in modules/mod_menu/tmpl/default.php around line 23

change

Code: Select all

	if ($item->id == $active_id) {
		$class .= 'current ';
	}

	if (in_array($item->id, $path)) {
		$class = 'active ';
	}

to

Code: Select all

	if ($item->id == $active_id) {
		$class .= 'current' . $params->get('class_sfx') . '"';
	}

	if (in_array($item->id, $path)) {
		$class = 'active' . $params->get('class_sfx') . '"';
	}

[Webdongle]

Just a few minor points

Would it be better to create a Template override than just edit a core file ?

Does
$this->escape code
need to be in there somewhere ?

[teapotbag]

Yeah I turned it into an override just now, I was just about to go back to edit to say that.

I have no idea about $this->escape, I really don't get much php, I copied that string of code like a dummy and well, it works great!

[Webdongle]

$this->escape
Is something to do with preventing SQL injection me thinks. It is included in a lot of default.php overrides.

It appears in
templates/beez5/html/com_content/article/default.php

Not sure if it needs to be in the modules override ?

Was hoping someone who new the answer would confirm the 'safety' of the code you copied.

[teapotbag]

woops!

You may be right, I've no idea.

Ok... anyone know this?

Think I'll remove the "solved" tick from the thread for now...