208305 - After stylesheet switched off and on, table-cell no longer creates the pseudo row

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect)

Description

[Jan Pazdziora]

User-Agent:       Mozilla/5.0 Galeon/1.2.10 (X11; Linux i686; U;) Gecko/20030314
Build Identifier: Mozilla/5.0 Galeon/1.2.10 (X11; Linux i686; U;) Gecko/20030314

Mozilla nicely renders four sibling elements (div's or ul's) with display:
table-cell in stylesheet as one row, presumably by creating the anonymous
"table-row" object. When I switch the stylesheet off, the page is correctly
rendered using the div or ul default display. However, when I switch the
stylesheet back on, those elements are no longer redisplayed in one line,
instead they are stacked verticaly.

Reproducible: Always

Steps to Reproduce:
1. Display the page with a stylesheet.
2. Turn the stylesheet off, using View -> Use Style -> Basic Page Style.
3. Be happy that the styling is properly gone.
4. Turn the stylesheet back on, using View -> Use Style -> The default stylesheet.
5. Observe that the result is not the same as when the page was first loaded.

Actual Results:  
The elements with display: table-cell are stack on top of each other.

Expected Results:  
I expected the elements to be rendered the same way as they were when the page
was loaded (or reloaded -- reload makes the page alright again), next to each
other horizontaly.

Comment 1

[Jan Pazdziora]

Attachment: The HTML file on which I tested multicolumn rendering using table-cell.

Comment 2

[Jan Pazdziora]

Attachment: The CSS file that work well first time the HTML is loaded but not after the stylesheet was switched off and on.

Comment 3

[Jan Pazdziora]

Attachment: The HTML file that loads with the stylesheet.

This is HTML file with a stylesheet href that will (hopefully :) load the
stylesheet directly from bugzilla.

Comment 4

[Jim Dabell]

I'm observing this behaviour on Mozilla 1.4-rc1 and Firebird 0.6.

Comment 5

[Boris Zbarsky [:bzbarsky]]

This is basically bug 121142... leaving as a separate bug for now, but marking
dependency

Comment 6

[Bernd]

Attachment: testcase

Comment 7

[Bernd]

Boris, I would expect that in the testcase that I attached the javascript would
change the display type for all three divs. Or is the javascript wrong?

Comment 8

[Boris Zbarsky [:bzbarsky]]

No, the JS is correct.  Probably the table pseudo-frames are breaking things
(see bug 162063 -- basically, the two blocks are each ending up in anonymous
table cells under the anonymous row that never got removed).

Comment 9

[Boris Zbarsky [:bzbarsky]]

Bug 162063 fixed Bernd's testcase.  Bug 148810 fixes the original testcase on this bug.

Comment 10

[Boris Zbarsky [:bzbarsky]]

Fixed by checkin for bug 148810.  Added tests.