84923 - neverwinternights.com - [LAYER]Neverwinter night FAQ un-readable (bad javascript?)

Status: RESOLVED FIXED

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

Description

[Rick Niles]

This maybe a legit Javascript bug, but I'm betting it's
a Evangelism bug of invalid Javascript.  However, I really
think these guys would be more likely to fix their broken
Javascript that say http://www.sony.com (grrr..)

Can someone figure out which it is and send a e-mail
the webmaster if it really is broken Javascript?

Comment 1

[Michael Hearn]

This is due to non-standards complient code methinks. It detects either NS4 or
IE4 but not apparently anything above. Setting to evangelism

-mike

Comment 2

[Phil Schwartau]

Layers are no longer supported. See bug 50711 for further details.
Reassigning to Evangelism component.

Here is a sample of the non-W3C-compliant HTML at this site:

<script language="Javascript">

 var bName = navigator.appName;
 var bVer = parseInt(navigator.appVersion);
 var NS4 = (bName == "Netscape" && bVer >= 4);
 var IE4 = (bName == "Microsoft Internet Explorer" && bVer >= 4);
 var NS3 = (bName == "Netscape" && bVer < 4);
 var IE3 = (bName == "Microsoft Internet Explorer" && bVer < 4);
 
// LAYER SWITCHING CODE
if (NS4 || IE4) {
 if (navigator.appName == "Netscape") {
  layerStyleRef="layer.";
  layerRef="document.layers";
  styleSwitch="";
  }else{
  layerStyleRef="layer.style.";
  layerRef="document.all";
  styleSwitch=".style";
 }
}

//Show layers
var onLayer = ""
function showLayer(layerName){
if (NS4 || IE4) {

    hideLayer(onLayer)
 }
 if (NS4 || IE4) {
  eval(layerRef+'["'+layerName+'"]'+styleSwitch+'.visibility="visible"');
 }  else {
  window.location.hash = layerName + "H"
 }
 onLayer = layerName 
}

//Hide Layer
function hideLayer(layerName){
 if (layerName != "") {
   if (NS4 || IE4) {
   eval(layerRef+'["'+layerName+'"]'+styleSwitch+'.visibility="hidden"');
  }
 } 
}

</script>

Comment 3

[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 4

[Bob Clary [:bc] (inactive)]

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

Comment 5

[Bob Clary [:bc] (inactive)]

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

Comment 6

[Christopher Aillon (sabbatical, not receiving bugmail)]

*** Bug 105100 has been marked as a duplicate of this bug. ***

Comment 7

[Doron Rosenberg (IBM)]

they moved over to Hiermenus, fixed!