CSS WG has agreed to modify the track sizing algorithm to include a new step: https://drafts.csswg.org/css-grid/#algo-stretch We used to do the stretch of the "auto" tracks at the end of the track sizing algorithm, however the change integrates this into the algorithm itself as the last step. See: https://github.com/w3c/csswg-drafts/issues/1150
Created attachment 320547 [details] Patch
Comment on attachment 320547 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=320547&action=review > Source/WebCore/rendering/GridTrackSizingAlgorithm.cpp:1077 > + std::optional<LayoutUnit> currentFreeSpace = freeSpace(m_direction); auto currentFreeSpace
Created attachment 320557 [details] Patch
Thanks for the review. (In reply to Sergio Villar Senin from comment #2) > Comment on attachment 320547 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=320547&action=review > > > Source/WebCore/rendering/GridTrackSizingAlgorithm.cpp:1077 > > + std::optional<LayoutUnit> currentFreeSpace = freeSpace(m_direction); > > auto currentFreeSpace Ack.
Comment on attachment 320557 [details] Patch Clearing flags on attachment: 320557 Committed r221931: <http://trac.webkit.org/changeset/221931>
All reviewed patches have been landed. Closing bug.
<rdar://problem/34693473>