149380 - gegsn.com site uses document.layers and document.all

Status: RESOLVED INVALID

(Tech Evangelism Graveyard :: English US, defect)

Description

[James Paige]

https://www.gegsn.com:443/CBSI/Sup_APeServices/default.asp?CBI=ACE?SM_PROXYTEMP=1
(General Electric Global Supplier Network) uses browser-specific code that
depends on support for document.layers or document.all

You will not be able to view the sample URL I have posted, because it requires a
username and password.

I encountered the bad code in the "AP eServices" web app in the "GE Aircraft
Engines" section of the site (once again, you cant get there without a login)

The bad code seems to be involved with putting a menu-bar at the top of the
page. I viewed the site in IE 5.x and was very puzzled why they didn't just do
this menu-bar in plain-ol HTML, as it doesnt seem to pop-up, andimate, make
french-fries, or any of the other garbage that is usually associated with
document.layers/document.all style dymanic menus.

Here is a small excerpt of their javascript. The page is riddled with this sort
of detection:

if (document.layers){//check for Netscape
        divWait = window.parent.document.layers["wait"];
        divMenuBar = window.parent.document.layers["MenuBar"];
        if (typeof(divMenuBar)!="undefined"){
                divWait.visibility = 'visible';
                divMenuBar.visibility='hide';
        }
}
//if IE then use this code
else {
        divWait = window.parent.document.all.item("wait");
        divMenuBar = window.parent.document.all.item("MenuBar");
        if (typeof(divMenuBar)!="undefined"){
                divMenuBar.style.visibility="hidden";
                divWait.style.visibility="visible";
        }
}

Comment 1

[Bob Clary [:bc] (inactive)]

tech evang june 2003 reorg

Comment 2

[James Paige]

gegsn.com does not seem to exist anymore.

marking INVALID