103533 - Cannot hide table that has attribute style

Status: RESOLVED WORKSFORME

(Core :: Layout: Tables, defect, P3)

Description

[José Jeria]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.4+)
Gecko/20011003
BuildID:    20011003

Cannot hide table that has inline css (style-attribute). 
See testcase, 2 tables in that document. Both should be hidden.
The first one is hidden, because the css is not set with the style-attribute.

Reproducible: Always
Steps to Reproduce:
1. Run testcase

Comment 1

[José Jeria]

Attachment: See the bug with this testcase

Comment 2

[Boris Zbarsky [:bzbarsky]]

I see this on Linux with build 2001-10-03-08 as well.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Oh, replacing <table> with <div> makes the bug disappear.

Comment 4

[karnaze (gone)]

In the table with style=, during painting of the table, the style visibility is 
visible. I don't think table code is changing visibility anywhere, so maybe 
there are some style rules that are doing it involving tables. -->pierre

Comment 5

[Pierre Saslawsky]

It is a regression from the rule tree landing (bug 91672).  Apparently we can no 
longer change by script the style of tables that have a style attribute.

Comment 6

[Pierre Saslawsky]

Bug 90960 could be related to this one.

Comment 7

[José Jeria]

Noticed that the second table actually is hidden is you hide it using:
document.getElementById("b").setAttribute("style","visibility:hidden;");

instead of:
document.getElementById("b").style.visibility = "hidden";

Dont know if this helps...

Comment 8

[Kevin McCluskey (gone)]

Moving to Mozilla1.1. Engineers are overloaded with higher priority bugs.

Comment 9

[José Jeria]

This works for me with build 20020219 Windows 2000. :-)

Comment 10

[Boris Zbarsky [:bzbarsky]]

Yep.  This got fixed sometime between Feb 8 and Feb 17.  Someone brave could try
to figure out exactly why... :)

worksforme on linux build 2002-02-20-08 (and 02-17-21) as well.

Comment 11

[Boris Zbarsky [:bzbarsky]]

This got fixed between 2002-02-15-08 and 2002-02-16-21, to be exact.

There are changes to TableLayoutStrategy in that range, as well as changes to
the CSSFrameConstructor's construction of table stuff, changes to the parser
affecting tables, all in that range. I  bet the CSSFrameConstructor is
responsible.