69246 - Does not properly display 2nd line of menu at Uptimes.net

Status: VERIFIED DUPLICATE of bug 66202

(Tech Evangelism Graveyard :: English US, defect)

Description

[Jerry Rowe]

When the mouse is over "Stats" on the first line, the 2nd line should have
"personal" as the rightmost item. Instead it displays "login".
The status line at the bottom does show that 'personal' is the choice and
activates the correct choice.

Comment 1

[Phil Schwartau]

This is due to a slight mistake in the HTML at the site. 
The site uses good browser-sniffing and detects Mozilla/N6 successfully:

var ns4 = (document.layers) ? 1:0;
var ie4 = (document.all)    ? 1:0;
var w3dom = (document.getElementById && document.getElementsByName);
if (ie4) w3dom = 0;
if (ns4) w3dom = 0; 



However, compare the NS4, IE4, and Mozilla codepaths for the MouseOver event:

function hi2(layer,imgnr) 
{
 if (w3dom) {document.getElementsByName("img" + layer + imgnr).item(0).src = 
eval("img" + imgnr + "on.src"); }

 if (ns4) {document.layers["menu"+layer].document["img" + layer + imgnr].src = 
eval("img" + layer + imgnr + "on.src"); }

 if (ie4) {document.all["menu" + layer].document["img" + layer + imgnr].src = 
eval("img" + layer + imgnr + "on.src"); }

}



Notice the "w3dom" branch has:       eval("img" + imgnr + "on.src")
but the "ns4", "ie4" branches have:  eval("img" + layer + imgnr + "on.src");



This means that in Mozilla, the wrong image is retrieved for the highlight.
For example, the "Personal" element mentioned by the reporter is as follows:


<a href="bottom.html?show=spersonal" target="bottom" onMouseOver="hi2(2,4);" 
                               etc.


When hi2(2,4) is called, NS4 and IE4 are directed to fetch "img24on.src"
But Mozilla is directed to fetch "img4on.src", which is a "login" gif.

Comment 2

[Phil Schwartau]

Attachment: Reduced HTML testcase

Comment 3

[Phil Schwartau]

If you try the reduced HTML testcase above, you will see the problem the 
reporter describes. However, there are no errors in the JavaScript Console.
Furthermore, if you save the testcase locally, and correct the mistake
described above, Mozilla functions the same as NS4 and IE4.


I'm reassigning this to Browser-General: what do we do in a case like this?
The HTML at the site does not make improper use of layers, and the 
browser-sniffing is perfectly correct. It's just a very small coding error...


Would this go to the Evangelism component?  Thanks - 

Comment 4

[Phil Schwartau]

Attachment: Dynamic reduced HTML testcase

Comment 5

[Phil Schwartau]

Try either testcase in NS4, IE4, and Mozilla and see the difference. 
With the dynamic testcase, you can manually correct the coding error 
(in the Mozilla case) by going to the textboxes and replacing 


                "img4on.src"  with "img24on.src", 
                "img4off.src" with "img24off.src"


(Only for Mozilla; in NS4 and IE4 the correct images are loaded alreay -)
Hit the "Execute" buttons to see the effect - 

Comment 6

[Asa Dotzler [:asa]]

over to evangelism.  would be nice to help since they're trying to do the right
thing.

Comment 7

[Blake Ross]

Reassigning evangelism bugs to [e-mail].

Comment 8

[Bob Clary [:bc] (inactive)]

*** This bug has been marked as a duplicate of 66202 ***

Comment 9

[Ian Tommins ('thelem')]

Verifying. Its really a RESOLVED FIXED, but it is not worth the spam.

Comment 10

[Bob Clary [:bc] (inactive)]

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