114051 - qxl.com - Hovering causes JavaScript errors instead of pop-up menu

Status: VERIFIED INVALID

(Tech Evangelism Graveyard :: Other, defect, P4)

Description

[Karl Johan Kleist]

Build ID: 2001 12 06 03. Windows 2000.

When hovering over an item in the list "Sök i underkategorier"
(the upper left lightblue area),
a pop-up menu is supposed to be displayed. This works fine in IE 5.5.

However, in Mozilla nothing happends except for errors in the JavaScript console:

- - -
Error: event has no properties
Source File: http://www.qxl.com/isroot/html/jscript/menu_maker_ie.js
Line: 348

Error: areCreated is not defined
Source File: http://www.qxl.com/isroot/html/jscript/menu_maker_ie.js
Line: 545
- - -

Since also Opera 5.12 fails to display the menus, I suspect IE centric (i.e.
buggy) JavaScript code. However, I'd like you Mozilla engineers to confirm this
before I nudge the webmaster. There is a small possibility that Mozille is to
blame, I guess.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Yup.  The site uses IE-specific event access methods in event handlers.

IE creates a global "window.event" object when a handler is called.  This is
accessible in the handler.  Mozilla, instead calls the handler function with a
the event in the variable called event.

In this case the handler is:

function (event) {
  popUp('elMenu7',event)
}

So the event object gets passed to the PopUp function in both browsers.
Now we have:

function popUp(menuName,e){
	if (!isLoaded) return;
	linkEl = event.srcElement;
	if (clickStart) linkEl.onclick = popMenu;
	if (!beingCreated && !areCreated) startIt();
	linkEl.menuName = menuName;   
	if (!clickStart) popMenu(e);
}

Notice that the event object is passed in to this function as the variable "e"
but the author ofthis function then turns around an uses the global window.event
object instead of "e", and that object is undefined in non-IE browsers.

Comment 2

[Tristan Nitot]

don't see any pop-ups with 5.0. Moz 097 and N621. Boris: can you confirm ?

Can't find any contact details.

Setting P4.

Comment 3

[Karl Johan Kleist]

Reporter here.

It seems that QXL have modified their auktion site as to no longer use
JavaScript pop-up menus. This may be a result of my pestering them for a while.

Resolving as invalid.

Comment 4

[z-aillon.c-obsolete2]

No menus - no bug.  verifying

Comment 5

[Bob Clary [:bc] (inactive)]

move tristan's resolved euro west bugs to other