Closed
Bug 1252000
Opened 9 years ago
Closed 8 years ago
[css-grid] Wrong computed values for grid-template-rows/columns if fragmentation grows some rows
Categories
(Core :: Layout, defect, P4)
Core
Layout
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)
References
Details
(Keywords: testcase-wanted)
Follow-up to bug 1144096.
Per the Grid spec, rows can grow during fragmentation:
"3. If a grid area’s size changes due to fragmentation (do not include items that span rows in this decision), increase the grid row as necessary for rows that either:
* have a content min track sizing function.
* are in a grid that does not have an explicit height and the grid row is flexible. "
https://drafts.csswg.org/css-grid/#fragmentation-alg
Bug 1144096 implements that, but that step after the step that calculates
the grid-template-rows/columns values that are cached for CSSOM.
Assignee | ||
Comment 1•8 years ago
|
||
Brad, I think this bug was fixed by your changes at some point, right?
(i.e. we collect all values needed for the computed value properties
and Grid API after the ReflowChildren() call)
Flags: needinfo?(bwerth)
Comment 2•8 years ago
|
||
Yes, barring a testcase that shows otherwise, it seems the code handles this correctly. ReflowChild called at https://dxr.mozilla.org/mozilla-central/source/layout/generic/nsGridContainerFrame.cpp#5731 and the building of the computed data happens shortly after at https://dxr.mozilla.org/mozilla-central/source/layout/generic/nsGridContainerFrame.cpp#5760.
Flags: needinfo?(bwerth)
Assignee | ||
Comment 3•8 years ago
|
||
OK, great.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: in-testsuite?
Keywords: testcase-wanted
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•