Bug 233196

Summary: [css-flexbox] Definiteness evaluation should use used flex-basis instead of specified
Product: WebKit Reporter: Sergio Villar Senin <svillar>
Component: Layout and RenderingAssignee: alan <zalan>
Status: NEW    
Severity: Normal CC: ahmad.saleem792, bfulgham, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar, WPTImpact
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Sergio Villar Senin
Reported 2021-11-16 09:27:26 PST
There are some tests failing because of that: * imported/w3c/web-platform-tests/css/css-flexbox/percentage-heights-016.html * imported/w3c/web-platform-tests/css/css-flexbox/percentage-heights-017.html * imported/w3c/web-platform-tests/css/css-flexbox/percentage-heights-018.html
Attachments
Patch (9.19 KB, patch)
2026-04-17 09:57 PDT, alan 		no flags
DetailsFormatted DiffDiff
Patch (11.04 KB, patch)
2026-04-17 10:59 PDT, alan 		no flags
DetailsFormatted DiffDiff
Patch (13.88 KB, patch)
2026-04-17 18:48 PDT, alan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Sergio Villar Senin
Comment 1 2021-11-18 02:25:22 PST
Note that this is controversial. I've filled [1] against the CSS specs in order to clarify it. [1] https://github.com/w3c/csswg-drafts/issues/6822
Radar WebKit Bug Importer
Comment 2 2021-11-23 09:28:22 PST
<rdar://problem/85707621>
Ahmad Saleem
Comment 3 2022-10-27 13:56:40 PDT
Safari Technology Preview 156 still fail following tests along with percentage-heights-014.html https://wpt.fyi/results/css/css-flexbox?label=master&label=experimental&aligned&view=subtest&q=percentage-height
alan
Comment 4 2026-04-17 09:57:28 PDT
Created attachment 479149 [details] Patch
alan
Comment 5 2026-04-17 09:58:38 PDT
*** Bug 312574 has been marked as a duplicate of this bug. ***
alan
Comment 6 2026-04-17 10:59:14 PDT
Created attachment 479153 [details] Patch
alan
Comment 7 2026-04-17 18:48:07 PDT
Created attachment 479159 [details] Patch
Note You need to log in before you can comment on or make changes to this bug.