Bug #15521
closed
Navigation "Icons in top frame" wrong rendering
Added by anoop atre over 18 years ago.
Updated almost 18 years ago.
Description
I noticed that in 4.0* the icons in top frame navigation broke in IE. On checking the html code and then in alt_topmenu_dummy.php I noticed that it was missing td tags.
Patch will be attached, used today's nightly from kraftb source to generate diff.
Not sure if this is how it should be fixed, core devs can decide.
(issue imported from #M2415)
Files
I'm not sure if it matters, but 3.8.0 seems to use 3 styles: class="c-menu", "c-admin" and "c-logout" (whereas the patch only uses class="c-admin").
By the way: note that adding the TD's also re-enables the different background color and border for the active menu, which makes navigating much easier.
if ($iconMenuMode) {
$contentArray0[] = '<td class="c-menu">'.$alt_menuObj->topMenu($loadModules->modules,0,'',3).'</td>';
if ($BE_USER->isAdmin()) {
$contentArray1[] = '<td class="c-admin">'.$alt_menuObj->adminButtons().'</td>';
}
$contentArray2[] = '<td class="c-logout">'.$alt_menuObj->topButtons().'</td>';
}
You have used wrong class names in your patch. Attached is a fix which will go to CVS.
Also available in: Atom
PDF