130887 - javascript "display: none" not working with tables

Status: VERIFIED DUPLICATE of bug 92868

(Core :: DOM: CSS Object Model, defect)

Description

[Gabriel Radic]

The function below is supposed to switch on/off the display of an element. It
work for most element types (div, tbody, others) but not for table. 

function hideshow(which) {	
	//shows or hides a div, used for info pannels and tree
	obj = document.getElementById(which);
	if ((obj.style.display == null) || (obj.style.display == ""))
		obj.style.display="none"
	else
		obj.style.display=""
}

Reproducible: Always

Steps to Reproduce:
1. Create an html document on a text editor.
2. Insert a <script> and paste the function
3. Insert a table, e.g. eg <table id="rockme" style="display:none">.
4. Outside the table tag, insert a link with href="javascript:hideshow('rockme');"
5. Open the page in Mozilla, click the link.


Actual Results:  The table will be showed, but re-clicking the link will not
hide it.

Expected Results:  clicking the link should switch the table display on/off.

I am on instant messenger
    ICQ - 5671715
    Yahoo - gabrielradic
    MSN - [e-mail]

Comment 1

[Phil Schwartau]

Attachment: HTML testcase

Comment 2

[Phil Schwartau]

Confirming bug; Mozilla trunk binaries 20020311xx WinNT, Linux.
You can toggle the <DIV> element back and forth ad infinitum,
but you can only toggle the <TABLE> element twice -

Browser, not engine ---> DOM Style
OS: Win2K ---> All

cc'ing Boris, jkeiser -

Comment 3

[Phil Schwartau]

Note: the testcase passes in IE6. You can toggle each element ad infinitum.

Comment 4

[Phil Schwartau]

Comment on attachment 74120 [details]
HTML testcase

Original testcase has a minor error; obsoleting it

Comment 5

[Phil Schwartau]

Attachment: HTML testcase (corrected)

Comment 6

[Boris Zbarsky [:bzbarsky]]

This got fixed on the 12th by bug 92868.  Worksforme in a March 14 build (though
fails in a March 11 build as Phil describes).

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

Comment 7

[Phil Schwartau]

Marking Verified - the testcase works fine with a trunk build from today,
2002-03-14, on WinNT.

Thanks, Boris!