Bug #15521
closedNavigation "Icons in top frame" wrong rendering
0%
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
Updated by Palahala over 18 years ago
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>';
}
Updated by Michael Stucki over 18 years ago
You have used wrong class names in your patch. Attached is a fix which will go to CVS.