147218 - Javascript:show() isn't working

Status: RESOLVED INVALID

(Core :: DOM: Core & HTML, defect)

Description

[Walter Ebeling]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.0rc2) Gecko/20020510
BuildID:    2002052306

Moving the mouse above a hint in a web page isn't showing the hint. Works in
IE5.5 and NS4.77.

Reproducible: Always
Steps to Reproduce:
start test.html testcase

Actual Results:  JS Debugger is showing bugs

Expected Results:  Page should work as designed

Comment 1

[Walter Ebeling]

Attachment: Zip file containing testcase, start test.html

Comment 2

[Alfonso Martinez]

What's this attachment?
I tried to change it to text/html but it only displays as garbage.

Comment 3

[Walter Ebeling]

The testcase is in a zip file. Mozilla is messing up the extension, see bug 
120327.... Rename it after downloading to zip, umpack the folder test and start 
test.html. 

Comment 4

[Phil Schwartau]

Reassigning to DOM Level 0 component; this is not a JS Engine issue -

Comment 5

[Phil Schwartau]

Here is the problem:

<div id="hint" style="position:  etc. etc."></div>

<a href="JavaScript:show(hint,hint1,400);"
   onmouseover="show(hint,hint1,400);"
   onmouseout ="hide(hint); return true;">


You are using |hint| in a way that only works in IE.

The W3C-standard way to access an HTML element with id="hint" is to
do |document.getElementById("hint")|. By using the id without any
qualification, as above, you are expecting that the HTML id has
somehow become a global variable in the JavaScript for the page.

This works in IE, because IE automatically puts every HTML id into the 
JavaScript namespace, so you can just say |hint| instead of 
|document.getElementById("hint")|.

This is not W3C-standard, however, and Mozilla does not support this. 
Have to mark this one invalid, because Mozilla is following the standard,
and the HTML here is relying on IE-only syntax.

The way to proceed would be to have an explicit declaration in JavaScript:

          var hint = document.getElementById("hint");


But I still see problems with your script. Here is the show() function:

function show(objeto,mensaje,width)
{
  tabla="<table  etc. etc."

  if(isNetscape)
  {
    objeto.document.write(tabla);
    objeto.document.close();

             etc.


In Mozilla, <div> elements don't have a |document| property.
But I can confirm that your page works in NN4.7, so I'm a little
puzzled by that. In Mozilla, you should do this instead:

                  hint.innerHTML = tabla

Comment 6

[Phil Schwartau]

Sorry, inside the function that would be:

                   objeto.innerHTML = tabla