Project

General

Profile

Actions

Bug #15521

closed

Navigation "Icons in top frame" wrong rendering

Added by anoop atre almost 19 years ago. Updated over 18 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2006-01-27
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.0
PHP Version:
4
Tags:
Complexity:
Is Regression:
Sprint Focus:

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

bugs_2415_topiconsmenu.patch (849 Bytes) bugs_2415_topiconsmenu.patch Administrator Admin, 2006-01-27 21:01
bug_2415.diff (2 KB) bug_2415.diff Administrator Admin, 2006-03-21 20:12
Actions #1

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>';
}

Actions #2

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.

Actions #3

Updated by Michael Stucki over 18 years ago

Fixed in CVS.

Actions

Also available in: Atom PDF