Project

General

Profile

Actions

Bug #14716

closed

XMenu_Layers fails to position layers correctly in Firefox

Added by old_raimund_bau almost 19 years ago. Updated almost 13 years ago.

Status:
Closed
Priority:
Should have
Category:
Frontend
Target version:
-
Start date:
2005-05-04
Due date:
% Done:

0%

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

Description

XMenu_Layers fails to position layers correctly in Firefox. It seems to temporarily add some value to the offset. Once any link in the menu is clicked and the menu is reloaded, the menu is displayed correctly.

example source code:

page.includeLibs.gmenu_layers = media/scripts/gmenu_layers.php
temp.MENU = HMENU
temp.MENU.1 = GMENU_LAYERS
temp.MENU.1 {
wrap = <table width="80px" align="left" cellspacing=0 cellpadding=0 border=0 class="menu"> | </table>
leftOffset = 80
relativeToTriggerItem = 1
expAll=1
hideMenuWhenNotOver = 20
hideMenuTimer = 200
dontHideOnMouseUp = 1
freezeMouseover = 1
displayActiveOnLoad = 1
layerStyle = witdh:80px; position:absolute;visibility:hidden; left:80px; top:0px;

NO {
wrap = &lt;tr class=&quot;menu&quot;&gt;&lt;td class=&quot;menu&quot;&gt;|&lt;/td&gt;&lt;/tr&gt;
XY = 80, 22
backColor = #FFFFFF
10.offset = 0, 17
10 = TEXT
10.fontColor = black
10.text.field = title
10.fontSize = 11
10.fontFile = fileadmin/template/fonts/arial.ttf
}
RO< .NO
RO=1
RO {
20 = BOX
20.dimensions = 0,21,250,1
20.color = #003399
}
ACT < .NO
ACT = 1
ACT {
10.fontColor = #003399
}
ACTRO < .RO
ACTRO = 1
CURRO < .RO
CURRO = 1

}

temp.MENU.2 = GMENU
temp.MENU.2 {
wrap = <table width="80px" align="left" cellspacing=0 cellpadding=0 border=0 class="menu"> | </table>

NO {
wrap = &lt;tr class=&quot;menu&quot;&gt;&lt;td class=&quot;menu&quot;&gt;|&lt;/td&gt;&lt;/tr&gt;
XY = 80, 22
backColor = #FFFFFF
10.offset = 10, 17
10 = TEXT
10.fontColor = black
10.text.field = title
10.fontSize = 11
10.fontFile = fileadmin/template/fonts/arial.ttf
20 = BOX
20.dimensions = 0,21,1,1 |*| 0,0,1,22
20.color = #003399
}
RO< .NO
RO=1
RO {
10.fontColor = #003399
}
ACT < .RO
ACT = 1
CUR < .RO
CUR = 1

}
----
(issue imported from #M1055)


Files

gmenu_layers.php.diff.txt (766 Bytes) gmenu_layers.php.diff.txt Administrator Admin, 2005-11-17 00:31
Actions #1

Updated by Ansgar Brauner over 18 years ago

I found the Problem in line 242 of the file typo3/sysext/cms/tslib/media/scripts/gmenu_layers.php

If you remove the width:100%; height:100%; at the end of the line it works (for me). I don't know if this causes other problems with other menu configurations. The maintainer of the file named above should check this out.

I attached the diff output to this bug-report

Actions #2

Updated by Ansgar Brauner over 18 years ago

The problem is still alive in version 3.8.1

Actions #3

Updated by Ansgar Brauner over 18 years ago

corrected the format of the diff-file. Please excuse the wrong upload. This was my first diff :-)

Actions #4

Updated by old_raimund_bau over 18 years ago

Thank you so much!

It works on 5 of mine installations with different settings. I think this should really go into the next version! How are we going to make it happen?

Actions #5

Updated by Ansgar Brauner over 18 years ago

Try to send the Bugreport to one of the core developers. I dont know how the bug-tracking-system works. You opened the bug. maybe you have more options to control the bug-report.

Actions #6

Updated by Ansgar Brauner about 17 years ago

This Problem is still active in Version 4.0.2. Maybe someone can take a look at my patch.

Actions #7

Updated by Michael Stucki about 17 years ago

There must have been some reason why the style attribute was there. So before we apply this patch, we should be very sure that this doesn't break on any other system.

Can any one of you check this, please?

In any case, it's too late for 4.1 now...

- michael

Actions #8

Updated by Ansgar Brauner about 17 years ago

Is there any possibility to an CVS Log entry where we can find who edited this file?

I'm using the patched version without any Problems.

Actions #9

Updated by Oliver Hader about 17 years ago

The "width:100%, height:100%;" settings depend on revision 75 from 2003-12-19 and were introduced by Kasper. I also have problems using TMENU_LAYERS in IE 6.0. I had to work-around to use CSS "correctly".

I guess the "100%" should enforce to use the maximum of available space for the DIV element. This could be done nowadays with other CSS properties.

Actions #10

Updated by Gideon So almost 17 years ago

Anybody will commit this patch to the core?? It's very good to eliminate the height=100%

Actions #11

Updated by Tolleiv Nietsch almost 13 years ago

  • Status changed from Needs Feedback to Closed
  • Target version deleted (0)
  • TYPO3 Version changed from 3.7.0 to 3.8

Not relevant anymore in 4.5+

Actions

Also available in: Atom PDF