RESOLVED FIXED 283060
[css-masonry] Update Intrinsic Sizing Rows 003 Mix1 Expected / Ref Test Case 
https://bugs.webkit.org/show_bug.cgi?id=283060
Summary [css-masonry] Update Intrinsic Sizing Rows 003 Mix1 Expected / Ref Test Case
Brandon
Reported 2024-11-13 09:42:24 PST
Use Ahem font and add colors to work around font sizing issues. * Grid 1 Remove spans. Add extra hidden item to correctly handle min-content size. Add sizing for grid template columns, so the 5th item is correctly placed. * Grid 2 Remove spans. Add extra hidden item to correctly handle min-content size. Add sizing for grid template columns, so the 5th item is correctly placed. * Grid 4-5 Remove spans that were causing width issues. * Grid 6 Remove spans that were causing width issues. Remove extra 5 not found in the main test case. * Grid 7 - 9 Remove spans that were causing width issues. Remove hidden items that were adding an extra column Add sizing for grid template columns; the grid appears to overflow without it. Add height of 3ch to the first item, so the first row is correctly sized.
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2024-11-13 09:42:42 PST
<rdar://problem/139810909>
Brandon
Comment 2 2024-11-13 09:43:34 PST
Pull request: https://github.com/WebKit/WebKit/pull/36596
EWS
Comment 3 2024-11-18 11:08:11 PST
Committed 286740@main (a8303551d4db): <https://commits.webkit.org/286740@main> Reviewed commits have been landed. Closing PR #36596 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.