Bug 232667 - [css-grid] Apply content alignment to row estimation in step 1
Summary: [css-grid] Apply content alignment to row estimation in step 1
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
Keywords: InRadar
Depends on:
Reported: 2021-11-03 07:29 PDT by zsun
Modified: 2022-05-13 03:07 PDT (History)
11 users (show)

See Also:

Patch (5.88 KB, patch)
2021-11-03 07:43 PDT, zsun
no flags Details | Formatted Diff | Diff
Patch (5.88 KB, patch)
2021-11-09 05:38 PST, zsun
no flags Details | Formatted Diff | Diff
Patch (6.01 KB, text/plain)
2022-01-10 10:24 PST, zsun
ews-feeder: commit-queue-

Note You need to log in before you can comment on or make changes to this bug.
Description zsun 2021-11-03 07:29:06 PDT
... when  all rows have a definite max track sizing function, and the container has a definite block-size.
Comment 1 zsun 2021-11-03 07:43:39 PDT
Created attachment 443199 [details]
Comment 2 zsun 2021-11-09 05:38:03 PST
Created attachment 443684 [details]
Comment 3 Radar WebKit Bug Importer 2021-11-10 06:30:23 PST
Comment 4 zsun 2022-01-10 10:24:44 PST
Created attachment 448768 [details]
Comment 5 Darin Adler 2022-01-10 14:34:11 PST
Comment on attachment 448768 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=448768&action=review

> Source/WebCore/rendering/GridTrackSizingAlgorithm.cpp:1196
> +            if ((rawTrackSize.minTrackBreadth().isLength() && rawTrackSize.minTrackBreadth().length().isSpecified()) && rawTrackSize.hasFixedMaxTrackBreadth() && rawTrackSize.minTrackBreadth().length() == rawTrackSize.maxTrackBreadth().length())

Seems like we should have a way to do == on two GridLength without calling length() twice.
Comment 6 zsun 2022-05-13 03:07:07 PDT
Affected test


The submitted patch doesn't seem fixing the test failure. It seems passing with this patch in my local(GTK) run though.

Please feel free to take this bug as I can't foresee that I would have time on it in a short while.