89491 - nbc4.com - [LAYER]Javascript image rollovers don't work

Status: VERIFIED WONTFIX

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

Description

[abrody]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Macintosh; U; PPC; en-US; rv:0.9.2) Gecko/20010702
BuildID:    Mozilla/5.0 (Macintosh; U; PPC; en-US; rv:0.9.2) Gecko/20010702

Netscape 4.7 and Explorer both are able to give me menus when I put my cursor
over the tabs on the URL above. Mozilla gives me nothing.

Reproducible: Always
Steps to Reproduce:
1. Browse to http://www.nbc4.com/
2. Put cursor over Weathernet4 tab


Actual Results:  no menu drops down, nothing crashes.  It doesn't see the rollover

Expected Results:  A menu should drop down with the different parts of the
WeatherNet4 website

Comment 1

[Phil Schwartau]

Site relies on layers and insufficient browser detection. 
Here is an example:


<A HREF= etc. 
   onMouseOver="popUp('elMenu1','menuimg1',event)" 
   onMouseOut="popDown('elMenu1')">
<IMG etc.>
</A>


The function popUp() is in http://www.nbc4.com/javascript/hierMenus.js
function popUp(menuName,objImg,e)
{ 
  if (NS4 && NSresized) startIt(); 
  if (!isLoaded) return; 
  linkEl = (NS4) ? e.target : event.srcElement; 
  if (clickStart) linkEl.onclick = popMenu; 
  if (!beingCreated && !areCreated) startIt(); 
  linkEl.menuName = (IE4) ? menuName : "document."+menuName;       
  if (!clickStart) popMenu(e, objImg); 
}


The variables NS4 and IE4 are defined as follows in http://www.nbc4.com/ 

NS4 = (document.layers && !is_webtv);
IE4 = (document.all && !is_webtv); 


The variable clickStart is defined as follows:

if (isMenu) {

    etc.
    clickStart = false;    // when clickstart is false, 
                           // it uses mouseOver as the event
}


where the variable isMenu is defined as follows:

isMenu = (NS4 || (IE4 && !isMac) || (IE5 && isMac));

Comment 2

[Phil Schwartau]

document.all and document.layers both evaluate to false in Mozilla,
so this site is not going to do the popUp() function - 

document.all is IE-only; document.layersr was supported in NN4.x 
but not in Mozilla/N6. See bug 50711 for further references on this.

Reassigning to Evangelism component; setting Priority to 1 since
the site is from NBC -

Comment 3

[Phil Schwartau]

For completeness, I should also include this function:


function popMenu(e, objImg)
{ 
  if (!isLoaded || !areCreated) return true; 
 
  eType = (NS4) ? e.type : event.type; 
  if (clickStart && eType != "click") return true; 
  hideAll(); 
 
  linkEl = (NS4) ? e.target : event.srcElement; 
         
  currentMenu = eval(linkEl.menuName); 
  currentMenu.hasParent = false; 
  currentMenu.menuObj.treeParent.startChild = currentMenu; 
         
  if (IE4) menuLocBod = document.body; 

  if(objImg){
                
    if (IE4) {
      var imgContainer = eval('document.all.' + objImg);                        
      iLeft = getImagePageLeft(imgContainer);
      iTop = getImagePageTop(imgContainer) + imgContainer.height;
    }
    else 
    {
     iLeft = eval('document.' + objImg + '.x');
     iTop = eval('document.' + objImg + '.y') + eval('document.' + objImg + 
                                                                    '.height');
    }
  }

  xPos = (iLeft) ? iLeft : (currentMenu.menuLeft) ? currentMenu.menuLeft : (NS4) 
? e.pageX : (event.clientX + menuLocBod.scrollLeft); 
  yPos = (iTop) ? iTop : (currentMenu.menuTop) ? currentMenu.menuTop : (NS4) ? 
e.pageY : (event.clientY + menuLocBod.scrollTop); 
 
  currentMenu.moveTo(xPos,yPos); 
  //currentMenu.keepInWindow() 
  currentMenu.isOn = true; 
  currentMenu.showIt(true); 
 
  return false; 
}
 

Comment 4

[Bob Clary [:bc] (inactive)]

contacted 07/10/01 via [e-mail]. Sent standard webmaster letter.

Comment 5

[abrody]

Updated the version of software used to the bug report.  I ([e-mail]) do
not know if the bug is present on other operating systems or versions of the
browser. 

Comment 6

[Blake Ross]

Milestone reality check.

Comment 7

[Bob Clary [:bc] (inactive)]

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

Comment 8

[Bob Clary [:bc] (inactive)]

bumping target milestone for followups to 0.9.5.

Comment 9

[Bob Clary [:bc] (inactive)]

-> 0.9.6, need to follow up on these asap

Comment 10

[abrody]

I don't know about 0.9.6 which I couldn't find a build for Mac OS X, but the
0.9.4 build which I found was made October 4th still doesn't have this feature
fixed.

Comment 11

[Bob Clary [:bc] (inactive)]

abrody, evangelism bugs are not bugs against the browser. they are indications
that the site in question needs to upgrade their content. In this particular
case the site is using Navigator 4 only layer api and is not supported by
Mozilla. The target milestone indicates when we think they have had enough time
to fix their content and that we should recheck it to see if they have fixed it
yet. From your comment it is clear they have not done so.

Note they are using an old version of hierMenus.js v3.06a 990624 and need to upgrade

Error: imgSiz is not defined
Source File: http://www.nbc4.com/javascript/hierMenus.js
Line: 35

Comment 12

[Doron Rosenberg (IBM)]

site has now MM_ dreamweaver functions (3.0 version, evil). they now have a ns5 var.

Comment 13

[abrody]

Dreamweaver is very popular among the web page editing community.   I go to my
Mac users group meeting every Tuesday here in the Washington DC area and lots of
people use it for webpage editing.   Maybe what we need to do is alert the
makers of Dreamweaver that there is a feature you find too difficult to support
with your browser.   I just downloaded the 0.9.4 October 17th nightly build for
Mac OS X, still NBC4 doesn't appear to be using the rollovers your browser
understands.   And it isn't an Explorer only feature.  Netscape 4.7.8 supports
NBC4's rollovers, but considering Netscape 6 based their work on Mozilla,
Netscape 6.1 for X and 8.5 do not support it.   Surprise, surprise!   

Comment 14

[Doron Rosenberg (IBM)]

mozilla/netscape 6 supports only the w3c web standards. ns4/ie has non-standard
implementations. Dreamweaver 3 only supports ns4/ie, while dreamweaver 4+
supports the web standards. Thus, all they need to do is upgrade to the dw4
functions.

Comment 15

[Bob Clary [:bc] (inactive)]

contacted via email:

[e-mail]
[e-mail]
[e-mail]
[e-mail]

Comment 16

[Bob Clary [:bc] (inactive)]

still no reply-> DC->tvlusers->wontfix

Comment 17

[abrody]

Can't there at least be a componant of Mozilla which checks for Dreamweaver 3.0 code and 
attempts to run by it?  Remember, Mozilla is supposed to be the framework for the next 
upgrade of Netscape.  The fact it worked on Netscape before shouldn't mean it doesn't work 
on Netscape in the future.

Comment 18

[Bob Clary [:bc] (inactive)]

No we will not support layers, and unless you want to take this bug and try to
get a response from them leave it WONT FIX (which means they won't fix their
content).

Comment 19

[abrody]

I just checked this page again, and it seems their content has changed, to make this bug 
report nearly moot.   But still layers can be a very useful feature, in being able to provide 
floating menus from images.   The fact that at least they used this feature in the past, and 
that I've seen this feature elsewhere on the web, suggests it is something we should 
support.   What surprises me though is that Layers is part of the w3c spec, and yet you 
won't support it?   Aren't we about standards?

Comment 20

[Bob Clary [:bc] (inactive)]

layers are not a part of the standards. if they are please include a link to the
appropriate document on http://www.w3.org/

Comment 21

[Brant Gurganus]

Verified per instructions at http://mozilla-evangelism.bclary.com/evangelism.html