Open Bug 569645 Opened 14 years ago Updated 1 year ago

change interaction of 'height' on table cells with baseline alignment (and size of cell box)

Categories

(Core :: Layout: Tables, defect)

defect

Tracking

()

People

(Reporter: dbaron, Unassigned)

References

Details

(Keywords: css2)

Today the CSS working group resolved CSS 2.1 issue 26:
http://wiki.csswg.org/spec/css2.1#issue-26
in a way that is compatible with IE but not with our code, so we should change our behavior to match.

In particular, it says that the 'height' property on table cells does not change the height of the cell box, but only sets a minimum height on the row.  This means that:
 * we should stop increasing the height of the anonymous box inside table cells in response to the 'height' property. (which also allows us to eliminate one of the two sets of table cell vertical alignment code, IIRC)
 * we should replace that code with code that is part of row height calculation

The combination of the two changes should only change testcases that involve 'vertical-align: baseline'.

For more details, see:
http://wiki.csswg.org/spec/css2.1#issue-26
http://lists.w3.org/Archives/Public/www-style/2010Jun/0048.html
Actually, the code isn't as I remembered it, so this may just require an update to nsTableRowFrame::UpdateHeight.
Summary: change interaction of 'height' on cells with baseline alignment (and size of cell box) → change interaction of 'height' on table cells with baseline alignment (and size of cell box)
(I think I looked into this at one point and decided it was more complicated than I first thought.)
Severity: normal → S3

The severity field for this bug is relatively low, S3. However, the bug has 6 duplicates.
:dholbert, could you consider increasing the bug severity?

For more information, please visit auto_nag documentation.

Flags: needinfo?(dholbert)

The last needinfo from me was triggered in error by recent activity on the bug. I'm clearing the needinfo since this is a very old bug and I don't know if it's still relevant.

Flags: needinfo?(dholbert)
You need to log in before you can comment on or make changes to this bug.