[wpt-sync] Sync PR 46009 - Fix automatic min size handling of flex and grid items to match the spec.
Categories
(Core :: Layout: Flexbox, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox127 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 46009 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/46009
Details from upstream follow.
Emilio Cobos Álvarez <emilio@chromium.org> wrote:
Fix automatic min size handling of flex and grid items to match the spec.
This is a prerequisite to fix issue 40877677 and related issues because
otherwise overflow: clip stops applying min-size: auto.Both the grid and flex specs mention "scroll container" already, which
means that visible and clip should behave the same:
- https://drafts.csswg.org/css-grid/#min-size-auto
- https://drafts.csswg.org/css-flexbox/#min-size-auto
There's technically a difference between "is scroll container" (what the
spec says) and "overflow is not clip or visible", but the CSSWG already
resolved on just looking at the overflow value in:
https://github.com/w3c/csswg-drafts/issues/7714Blink was doing the right thing for flex only on replaced elements for
some reason. For grid, most stuff works already due to it using
ComputedStyle::IsScrollContainer, but there was one case missing, which
I added a test for.Bug: 40877677
Change-Id: I49f57b725ae7feb18250f1e40b3856ec2231bfb1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5505559
Reviewed-by: John Palmer \<jopalmer@google.com>
Reviewed-by: John Palmer \<jopalmer@chromium.org>
Reviewed-by: Morten Stenshorne \<mstensho@chromium.org>
Auto-Submit: Emilio Cobos Álvarez \<emilio@chromium.org>
Commit-Queue: Ian Kilpatrick \<ikilpatrick@chromium.org>
Reviewed-by: Ian Kilpatrick \<ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1295163}
Assignee | ||
Updated•5 months ago
|
Assignee | ||
Comment 1•5 months ago
|
||
Assignee | ||
Comment 2•5 months ago
|
||
CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 2 tests
Status Summary
Firefox
PASS
: 2
Chrome
FAIL
: 2
Safari
FAIL
: 2
Links
Comment 4•5 months ago
|
||
bugherder |
Description
•