1563050 - [css-contain] layout containment should suppress baseline alignment for table-cell

Status: RESOLVED FIXED

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

Description

[Daniel Holbert [:dholbert]]

Right now, we fail the WPT http://w3c-test.org/css/css-contain/contain-layout-baseline-004.html ( https://wpt.fyi/results/css/css-contain/contain-layout-baseline-004.html?label=master&label=experimental ).

Per the spec, layout containment is supposed to apply to table-cell (but not other internal table parts), which means it should suppress baseline alignment in this case. But it's apparently not suppressing it right now, per this testcase.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

Bugbug thinks this bug is a task, but please change it back in case of error.

Comment 2

[Daniel Holbert [:dholbert]]

Attachment: Bug 1563050: Treat table-cells with contain:layout as if they had no baseline. r?TYLin

This makes us pass WPT test contain-layout-baseline-004.html (i.e. it makes us
treat one table cell as having no baseline, so we align its bottom edge with
the other cell's baseline, as the test expects).

This is required by the spec text at [1] which excludes most table parts but
includes table cells, and says "the containing element is treated as having no
baseline").

[1] https://drafts.csswg.org/css-contain/#containment-layout

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/824b5c4ed05e
Treat table-cells with contain:layout as if they had no baseline. r=TYLin

Comment 4

[Daniel Holbert [:dholbert]]

I think this landed after the Beta69 branch point, so this will need a beta uplift in order for this to be fixed in the initial release of this feature, CSS Containment (which is targeting Firefox 69 release via bug 1487493).

I'll request beta uplift approval here after it's clear that the autoland landing has stuck.

Comment 5

[Daniel Holbert [:dholbert]]

Comment on attachment 9076288 [details]
Bug 1563050: Treat table-cells with contain:layout as if they had no baseline. r?TYLin

Beta/Release Uplift Approval Request

• User impact if declined: Possible broken layout (misaligned table cells in pages that use contain:layout)
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Not risky. This is just adding one line of logic to make us ignore a table-cell's actual baseline (as the spec says we should) when it has the CSS contain:layout applied to it.

I'm requesting uplift because it's trivial and because we're intending to ship the contain property with Firefox 69, so it'd be nice to have this fixed as part of that release.

• String changes made/needed: None

Comment 6

[Daniel Holbert [:dholbert]]

Just to be extra clear since it's merge week -- the beta-uplift-request is for Beta69, the "new" beta.

Comment 7

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/824b5c4ed05e

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9076288 [details]
Bug 1563050: Treat table-cells with contain:layout as if they had no baseline. r?TYLin

Minor CSS Containment fix. Approved for 69.0b4.

Comment 9

[Dorel Luca [:dluca]]

https://hg.mozilla.org/releases/mozilla-beta/rev/b38ee676c48b