Project

General

Profile

Actions

Bug #16277

closed

GMENU - IE fails to respond to mouse over when using XHTML strict

Added by Tristan Knapp almost 18 years ago. Updated almost 12 years ago.

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

0%

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

Description

In class.ts_lib_fe.php the generated Javascript code is probably not XHTML compatible. I was not able to conclusively figure out if this is a one of IE's "features" or is according to specification.

The following "works" however...

In the over / out js functions for the rollovers the IE check works under trans but not under strict.

In over/out
typeof(document.getElementById)=="function"

needs to be when using xhtml strict with IE in over/out
typeof(document.getElementById)=="object"

I am not familiar enough with the core to supply a proper patch.

There is a news group article http://lists.typo3.org/pipermail/typo3-english/2006-May/024050.html in which this was initially mentioned.
(issue imported from #M3729)


Files

0003729.patch (1.23 KB) 0003729.patch Administrator Admin, 2007-03-08 15:35
Actions #1

Updated by Marc Wöhlken over 17 years ago

Hi!
The problem is located at line 3648 and 3654 in file class.tslib_fe.php:
Instead of

else if (typeof(document.getElementById)=="function" && ...

it should be

else if ( (document.getElementById) && ...

Regards
Marc & Sven

Actions #2

Updated by Patrick Gaumond about 17 years ago

We had this exact problem and the fix worked.

Ernesto, can you make a patch for the 4.1 release ? (I can't...)

: (

Patrick

Actions #3

Updated by Martin Kutschker about 17 years ago

Interesting that typeof yields "object" in IE.

Actions #4

Updated by Markus Timtner about 17 years ago

This fix worked for me too.
Strange that it hasn't been put in 4.1.

Actions #5

Updated by Oliver Hader about 17 years ago

... because nobody managed yet to create a patch file. Now there is one! ;-)

Actions #6

Updated by Sudara Williams - almost 17 years ago

Hello!

I would just like to push a bit to get this included in the next release - it would have been nice to see it fixed in 4.1.1

Without this patch xhtml strict sites cannot use GMENUs with rollovers (while still supporting IE ;)

Danke!

Actions #7

Updated by Ernesto Baschny almost 17 years ago

Fix commited to trunk + TYPO3_4-1. So it should be in 4.1.2 and 4.2.x.

Actions #8

Updated by Chris topher almost 12 years ago

  • Category deleted (Communication)
  • Target version deleted (0)
Actions

Also available in: Atom PDF