display.style in Firefox

[tjk]

Hi all,

I'm having trouble with a display.style reference displaying correctly in Firefox (Mac, not sure about PC yet). The page calls a javascript function upon loading that is supposed to hide one of two tables on the page. Whilst it does hide the table as asked - there is an empty space where the table would have been.

My understanding is that if display.style = 'none'; then the surrounding page content should contract to fill the space of the hidden element. Is this correct?

My javascript function for the hidden effect is:

document.getElementById('table_id').display.style = 'none';

Has anyone come across this problem in Firefox before? It seems to work fine in Safari.

Thanks in advance
Tim

[AutisticCuckoo]

You've got it the wrong way round.

document.getElementById("table_id").style.display = "none";

You want to set the display CSS property of the style attribute of an element.

[tjk]

Sorry about that. It's been a longggg day.

Aside from my error....any ideas?

[AutisticCuckoo]

It should work. Can you provide a link to the page in question?

[tjk]

http://www.eventlist.com.au.

[AutisticCuckoo]

There are 44 validation errors on that page, some of which are quite serious. Always make sure that the markup is valid before you start looking for presentational issues.

You have a FORM element as an immediate child of a TABLE. Who knows how this mucks up the DOM tree and the rendering in Firefox?

Fix the validation problems and see if that helps. If not, post back and we'll see what it could be.

BTW, you should probably use an HTML doctype rather than XHTML. If not, you should learn about the differences between XHTML vs HTML. Remember that XHTML is case-sensitive, so you can't write attribute names as onLoad etc.

[tjk]

Thanks AutisticCuckoo. I'll work through your advice and see how it goes.

Much appreciated. Tim.

[bosko]

Hi!

I'm pc user. I think this is not browser issue. Try to use ...style.visibility = 'hidden'; instead style.display = 'none';
Visibility will just hide the table, not remove it.

[AutisticCuckoo]

@bosko: using visibility="hidden" will cause the very same effect that Tim is seeing now, and which is undesirable. He's right in using display="none", but he needs to fix the markup errors to make it work properly.

[SimplyFu]

visibility == hidden will leave space where it was occupied by the element. It might not be as intended.

[bosko]

You're right. I understood the problem in the opposite way....