| Summary: | [CSS Grid Layout] Fix missing layout in flexible and content sized columns | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Manuel Rego Casasnovas <rego> | ||||||||
| Component: | CSS | Assignee: | Manuel Rego Casasnovas <rego> | ||||||||
| Status: | RESOLVED FIXED | ||||||||||
| Severity: | Normal | CC: | commit-queue, esprehn+autocc, glenn, jfernandez, kling, kondapallykalyan, svillar | ||||||||
| Priority: | P2 | ||||||||||
| Version: | 528+ (Nightly build) | ||||||||||
| Hardware: | Unspecified | ||||||||||
| OS: | Unspecified | ||||||||||
| Bug Depends on: | |||||||||||
| Bug Blocks: | 133304 | ||||||||||
| Attachments: |
|
||||||||||
This has been already fixed in Blink and will be ported to WebKit: https://codereview.chromium.org/146833018 Created attachment 223965 [details]
Patch
Comment on attachment 223965 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=223965&action=review r=me > Source/WebCore/ChangeLog:8 > + From Blink r166914. Don't need to specify this if you're the original author. Created attachment 224346 [details]
Patch
Patch for landing.
Comment on attachment 224346 [details] Patch Clearing flags on attachment: 224346 Committed r164214: <http://trac.webkit.org/changeset/164214> All reviewed patches have been landed. Closing bug. |
Created attachment 223963 [details] Example file to reproduce the issue RenderGrid::logicalContentHeightForChild() is called for some items at the beginning of RenderGrid::layoutGridItems() from RenderGrid::computeUsedBreadthOfGridTracks(). This causes that the comparison inside the for loop in RenderGrid::layoutGridItems() does not detect width changes, so elements won't be marked as needsLayout. So the comparison is done in RenderGrid::logicalContentHeightForChild() and the element is marked to perform a layout if the width has changed. The issue can be reproduced easily with a simple grid with one flexible or content sized column, all the available width is not used. On top of that, when you resize the window the flexible or content sized columns are not updating their size properly.