[wpt-sync] Sync PR 31353 - [flex] Change ApplyStretchAlignmentToChild to use stretching.
Categories
(Core :: Layout: Flexbox, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox95 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 31353 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/31353
Details from upstream follow.
Ian Kilpatrick <ikilpatrick@chromium.org> wrote:
[flex] Change ApplyStretchAlignmentToChild to use stretching.
When we set a forced block-size on a child it'll be that size ignoring
all other constraints.This was problematic for tables, as we ignored the intrinsic block-size
of the tables.Previously we'd manually calculate the "stretched" cross-axis size for
items, with tables we didn't consider the intrinsic block-size.Here we skip the problem by just setting the stretch constraint on a
flex-item instead of the forced block-size.Eventually we'll be able to remove the:
FlexItem::cross_axis_size_
member.Bug: 1257572
Change-Id: I8824763e2fdb2c46328737a66c66ac3e4e3f279bReviewed-on: https://chromium-review.googlesource.com/3221603
WPT-Export-Revision: 094229f0855801847fe9afeea8d9c7a1c6e55fc4
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 1•3 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=7a9d939551b84f46dd15738c05be7495b97d7a06
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/18069986fb89 [wpt PR 31353] - [flex] Change ApplyStretchAlignmentToChild to use stretching., a=testonly
Assignee | ||
Comment 3•3 years ago
|
||
Test result changes from PR not available.
Comment 4•3 years ago
|
||
bugherder |
Description
•