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

NEW
Unassigned

Status

()

Core
Layout: Tables
7 years ago
2 years ago

People

(Reporter: dbaron, Unassigned)

Tracking

({css2})

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

7 years ago
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
(Reporter)

Comment 1

7 years ago
Actually, the code isn't as I remembered it, so this may just require an update to nsTableRowFrame::UpdateHeight.
(Reporter)

Updated

7 years ago
Duplicate of this bug: 634489
(Reporter)

Updated

7 years ago
Duplicate of this bug: 634489
(Reporter)

Updated

6 years ago
Keywords: css2
(Reporter)

Updated

6 years ago
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)
(Reporter)

Updated

5 years ago
Duplicate of this bug: 804227
Duplicate of this bug: 1040934
(Reporter)

Updated

3 years ago
Duplicate of this bug: 1103614
(Reporter)

Comment 7

3 years ago
(I think I looked into this at one point and decided it was more complicated than I first thought.)

Comment 8

2 years ago
I am adding these

http://www.gtalbot.org/BrowserBugsSection/css21testsuite/baseline-empty-table-cells-RobertHogan-1.html

http://www.gtalbot.org/BrowserBugsSection/css21testsuite/baseline-empty-table-cells-RobertHogan-2.html

hoping they are useful for this bug report.

Gérard
Duplicate of this bug: 1216164
Duplicate of this bug: 1211829
You need to log in before you can comment on or make changes to this bug.