[SOLVED]topmenu active

[tozi]

Hi all i need active class in some item eg.Sortiment, aktuality.

How to do it?

 

My web: http://presta.simplydesign.sk/

Edited May 4, 2014 by tozi (see edit history)

[vekia]

you mean that you want to add class="active" for these two <li> items?

[tozi]

Yes that correct.

[vekia]

how you added aktuality menu item?

[tozi]

In modul blocktopmenu. And i have presta 1.5.6.1.

 

Edited May 1, 2014 by tozi (see edit history)

[vekia]

in blocktopmenu.php you can find code:

case 'LNK':
					$link = MenuTopLinks::get((int)$id, (int)$id_lang, (int)$id_shop);
					if (count($link))
					{
						if (!isset($link[0]['label']) || ($link[0]['label'] == ''))
						{
							$default_language = Configuration::get('PS_LANG_DEFAULT');
							$link = MenuTopLinks::get($link[0]['id_linksmenutop'], $default_language, (int)Shop::getContextShopID());
						}
						$this->_menu .= '<li><a href="'.Tools::HtmlEntitiesUTF8($link[0]['link']).'"'.(($link[0]['new_window']) ? ' target="_blank"': '').'>'.$link[0]['label'].'</a></li>'.PHP_EOL;
					}
					break;

change it to:

				case 'LNK':
					$link = MenuTopLinks::get((int)$id, (int)$id_lang, (int)$id_shop);
					if (count($link))
					{
						if (!isset($link[0]['label']) || ($link[0]['label'] == ''))
						{
							$default_language = Configuration::get('PS_LANG_DEFAULT');
							$link = MenuTopLinks::get($link[0]['id_linksmenutop'], $default_language, (int)Shop::getContextShopID());
						}
						$this->_menu .= '<li class="'.($link[0]['id_linksmenutop']==2 || $link[0]['id_linksmenutop']==1 ? 'class="active"':'').'"><a href="'.Tools::HtmlEntitiesUTF8($link[0]['link']).'"'.(($link[0]['new_window']) ? ' target="_blank"': '').'>'.$link[0]['label'].'</a></li>'.PHP_EOL;
					}
					break;

[tozi]

Thanks Milos, but doesnt work correct. Look Aktuality and Sortiment.

<li class="class=" active""="">

Edited May 2, 2014 by tozi (see edit history)

[tozi]

Any idea?

[vekia]

oops my code is bad

instead of line:

						$this->_menu .= '<li class="'.($link[0]['id_linksmenutop']==2 || $link[0]['id_linksmenutop']==1 ? 'class="active"':'').'"><a href="'.Tools::HtmlEntitiesUTF8($link[0]['link']).'"'.(($link[0]['new_window']) ? ' target="_blank"': '').'>'.$link[0]['label'].'</a></li>'.PHP_EOL;

use

						$this->_menu .= '<li class="'.($link[0]['id_linksmenutop']==2 || $link[0]['id_linksmenutop']==1 ? 'active':'').'"><a href="'.Tools::HtmlEntitiesUTF8($link[0]['link']).'"'.(($link[0]['new_window']) ? ' target="_blank"': '').'>'.$link[0]['label'].'</a></li>'.PHP_EOL;

[tozi]

Milos, this modification completely knocked me add links through the module.   Thank you for trying. 

[vekia]

can you show how the link looks like at the moment?

[tozi]

Here http://presta.simplydesign.sk/

<li class="class=" active""=""><a href="http://presta.simplydesign.sk/index.php?fc=module&module=newsiw&controller=news">Aktuality</a></li>

[vekia]

hello

it looks the same as before

you changed code as i suggested?

 

btw. menu module needs to be recompiled, because it uses caching, moreover module contents is generated in php file (not in tpl as it is in other modules). THis is why it's worth to open module configuration page, and just hit save there, module will be regenerated with new code

[tozi]

You're right. It's working. Thanks a lot. 

[tozi]

One little problem. Selector "active" is active permanently.

[vekia]

and it shouldn't? 

when it should be applied?

[tozi]

No. I need to be added when the selector is clicked on an item in the menu.