85322 - supportwizard.com - Javascript-implemented mouseovers don't work

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: English US, defect, P3)

Description

[wesolows]

On the demo page above, enter the demo.  Click any of the buttons in the left
frame.  Nothing will happen.  The javascript console reports numerous errors.  I
 won't have a hard time believing this site uses bogus javascript, but this does
work in NN4.7x.

Javascript console output:

Error: parent.MENU_FRAME.document.MENU_FORM.submit is not a function
...
Error: document.all has no properties
Source File: http://www.supportwizard.com/
Line: 52
...

This is mozilla 0.8.1, 0.9, 0.9.1, and recent nightlies including 0611.  Good
odds it's never worked.

Comment 1

[Phil Schwartau]

Changing summary slightly: the links at the site do work in Mozilla.
It is the mouseovers above the links that do not work.

Your suspicions are correct; the site uses JavaScript that is not
W3C-compliant. Specifically, it uses layers, which have been deprecated.
See bug 50711 for further details on this topic. The mouseovers are coded 
in the HTML as follows:

<a href="#" onMouseOver="closeMenu()  etc. etc. 


The JavaScript for this is as follow:


var n = (document.layers) ? 1:0;

function showMenu(name) {
 closeMenu();
 if (n)  document.layers[name].visibility="show";
 else document.all[name].style.visibility="visible";
}

function closeMenu() {
 if (n)  {
  document.layers['menu1'].visibility="hide";
  document.layers['menu2'].visibility="hide";
  document.layers['menu3'].visibility="hide";
  document.layers['menu4'].visibility="hide";
  document.layers['menu5'].visibility="hide";
  document.layers['menu6'].visibility="hide";
  document.layers['menu7'].visibility="hide";
  document.layers['menu8'].visibility="hide";
 }
 else {
  document.all['menu1'].style.visibility="hidden";
  document.all['menu2'].style.visibility="hidden";
  document.all['menu3'].style.visibility="hidden";
  document.all['menu4'].style.visibility="hidden";
  document.all['menu5'].style.visibility="hidden";
  document.all['menu6'].style.visibility="hidden";
  document.all['menu7'].style.visibility="hidden";
  document.all['menu8'].style.visibility="hidden";
 }
}

Comment 2

[Phil Schwartau]

document.layers has been deprecated in Mozilla, so the variable 'n' 
above evaluates to false. This throws us in the document.all codepath.
However, document.all is Microsoft-only and not W3C. Hence the error
in the JavaScript Console:

              Error: document.all has no properties

Reassigning to Evangelism component - 

Comment 3

[Phil Schwartau]

Note: as for the other error you report, 

    Error: parent.MENU_FRAME.document.MENU_FORM.submit is not a function

I believe this must have come from another site you visited in Mozilla.
I cannot find any reference to these items at the site above -

Comment 4

[Bob Clary [:bc] (inactive)]

Arun, I'm giving you all the Linux bugs. If these bugs can be confirmed for
other OS's please (someone) change the Platform/OS to All/All and reassign to
the default Evangelism Component Owner.

Comment 5

[Bob Clary [:bc] (inactive)]

All Evangelism Bugs are now in the Product Tech Evangelism. See bug 86997 for
details.

Comment 6

[Bob Clary [:bc] (inactive)]

Arun, I am taking your Linux bugs minus banks. Hope you don't mind.

Comment 7

[Zach Lipton [:zach]]

Mass reassign of all tech-evangelism us general bugs assigned to bc to 
doron except bc's P1 bugs. You may search for this mass reassign (it is 
305 bugs) by searching for the keyword 'greeneggsandham'

Comment 8

[Kai Lahmann (is there, where MNG is)]

every JS got removed.

Comment 9

[Doron Rosenberg (IBM)]

v