Closed
Bug 1174569
Opened 10 years ago
Closed 9 years ago
[css-grid] Implement the Grid Sizing Algorithm
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla50
People
(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
21.21 KB,
patch
|
dholbert
:
review+
|
Details | Diff | Splinter Review |
Assignee | ||
Comment 2•9 years ago
|
||
I think this is good enough to implement Step 2 in the spec algorithm.
It actually fixes all of the testcases in the dependent bugs.
I'd like to land this for now so that people don't stumble on this
and can continue to find more bugs and differences we have compared
to other UAs.
Attachment #8761857 -
Flags: review?(dholbert)
Comment 3•9 years ago
|
||
Comment on attachment 8761857 [details] [diff] [review]
Use the resolved column sizes to guide intrinsic row sizing.
Review of attachment 8761857 [details] [diff] [review]:
-----------------------------------------------------------------
This looks good. Sorry for the delay! r=me
Attachment #8761857 -
Flags: review?(dholbert) → review+
Pushed by mpalmgren@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c719d640442b
[css-grid] Use the resolved column sizes to guide intrinsic row sizing. r=dholbert
Comment 5•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox50:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Comment 7•9 years ago
|
||
"12.1. Grid Sizing Algorithm" is implemented for the most part, only step 3
is missing. I filed bug 1300366 for the remaining work.
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Flags: in-testsuite+
Keywords: leave-open
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•