Closed (won't fix)
Project:
Menu Subtitle
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
20 Jun 2013 at 14:17 UTC
Updated:
21 Jun 2013 at 08:59 UTC
Actually (TWO LINK):
<LI>
<span>
<a href="">TITLE</a>
</span>
<span>
<a href="">SUB TITLE</a>
</span>
</LI>WITH MY VERSION (ONE LINK)
<LI>
<span>
<a href="">TITLE <BR /> <SPAN>SUBTITLE</SPAN></a>
</span>
</LI>____________
modules/menu_subtitle/menu_subtitle.module
Line 130 (Replace with this code) :
if (!empty($subtitle)) {
$output .= '<span class="menu-link-subtitle menu-link-subtitle-below">' . l($element['#title']."<br /><span>".$subtitle."</span>", $element['#href'], $element['#localized_options']) . '</span>';
} else {
$output .= '<span class="menu-link-title">' . l($element['#title'], $element['#href'], $element['#localized_options']) . '</span>';
}
____________
includes/modules/common.inc
Line 2420 / function l()
Change HTML => FALSE to TRUE
$options += array(
'attributes' => array(),
'html' => TRUE,
);
Comments
Comment #1
mecmartini commentedHi jackdaniel9,
I don't like the one link version, because it's needed a br tag. I think two links are better for CSS reasons. Anyway this is not a bug report, but a feature request ;)
Greetins
Comment #2
mecmartini commented