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

VERIFIED WONTFIX

Status

Tech Evangelism Graveyard
English US
P1
normal
VERIFIED WONTFIX
17 years ago
3 years ago

People

(Reporter: abrody, Assigned: bc)

Tracking

unspecified

Details

(Whiteboard: [no reply][needs new contact][aok], URL)

(Reporter)

Description

17 years ago
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

17 years ago
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

17 years ago
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 -
Assignee: rogerl → bclary
Status: UNCONFIRMED → NEW
Component: Javascript Engine → Evangelism
Ever confirmed: true
OS: MacOS X → All
Priority: -- → P1
QA Contact: pschwartau → zach
Hardware: Macintosh → All
Summary: Javascript image rollovers don't work → [LAYER]Javascript image rollovers don't work

Comment 3

17 years ago
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; 
}
 
(Assignee)

Comment 4

17 years ago
contacted 07/10/01 via nbc4dc@nbc.com. Sent standard webmaster letter.
Status: NEW → ASSIGNED
Summary: [LAYER]Javascript image rollovers don't work → nbc4.com - [LAYER]Javascript image rollovers don't work
(Reporter)

Comment 5

17 years ago
Updated the version of software used to the bug report.  I (abrody@smart.net) do
not know if the bug is present on other operating systems or versions of the
browser. 
OS: All → MacOS X
Hardware: All → Macintosh
Target Milestone: --- → mozilla0.9.2

Comment 6

17 years ago
Milestone reality check.
Target Milestone: mozilla0.9.2 → mozilla0.9.4
OS: MacOS X → All
Hardware: Macintosh → All
(Assignee)

Comment 7

17 years ago
All Evangelism Bugs are now in the Product Tech Evangelism. See bug 86997 for
details.
Component: Evangelism → US English
Product: Browser → Tech Evangelism
Version: other → unspecified
(Assignee)

Comment 8

17 years ago
bumping target milestone for followups to 0.9.5.
Target Milestone: mozilla0.9.4 → mozilla0.9.5
(Assignee)

Comment 9

17 years ago
-> 0.9.6, need to follow up on these asap
Target Milestone: mozilla0.9.5 → mozilla0.9.6
(Reporter)

Comment 10

17 years ago
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.
(Assignee)

Comment 11

17 years ago
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
Blocks: 85104
Status: ASSIGNED → NEW
Whiteboard: [no reply][needs new contact]
Target Milestone: mozilla0.9.6 → ---
site has now MM_ dreamweaver functions (3.0 version, evil). they now have a ns5 var.
(Reporter)

Comment 13

17 years ago
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!   
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.
(Assignee)

Comment 15

17 years ago
contacted via email:

nbc4dc@nbc.com
msnbc.tvsknbc@nbc.com
nbc4ny@nbc.com
ij.hudson@nbc.com
Status: NEW → ASSIGNED
Target Milestone: --- → Dec
(Assignee)

Updated

16 years ago
Whiteboard: [no reply][needs new contact] → [no reply][needs new contact][aok]
(Assignee)

Comment 16

16 years ago
still no reply-> DC->tvlusers->wontfix
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → WONTFIX
(Reporter)

Comment 17

16 years ago
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.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
(Assignee)

Comment 18

16 years ago
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).
Status: REOPENED → RESOLVED
Last Resolved: 16 years ago16 years ago
Resolution: --- → WONTFIX
(Reporter)

Comment 19

16 years ago
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?
(Assignee)

Comment 20

16 years ago
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

16 years ago
Verified per instructions at http://mozilla-evangelism.bclary.com/evangelism.html
Status: RESOLVED → VERIFIED
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in before you can comment on or make changes to this bug.