1222724 - [css-grid] Row with track size "auto" doesn’t expand to wrap content

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Šime Vidas]

Attachment: Capture.PNG

User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:42.0) Gecko/20100101 Firefox/42.0
Build ID: 20151029151421

Steps to reproduce:

In Firefox Nightly, open this demo: https://jsbin.com/duveso/edit?css,output


Actual results:

The 2. row does not expand vertically to wrap the <main> element. (See attached screenshot.)


Expected results:

The 2. row should expand. Open in Chrome (with Web Platform flag) to see desired result.

Comment 1

[Zéfling]

Attachment: A test case

Example of problem of sizing auto in width and height.

Comment 2

[yisibl]

In Firefox this problem is a headache.

I wrote a simplified example: http://jsbin.com/ligiyuluma/edit?html,css,output

Comment 3

[David Smith]

In a more general sense, it doesn't respect calculated values, as opposed to explicit values.

If you give a cell value a height, the entire row that it's a part of will be set to that height.  If you give it a min-height, the row will be set to that min height, but it won't expand beyond that if the cell grows further.

If a component of a cell (such as an image thumbnail) has width/height defined with a calc() function, the height of the cell will be based on the original image size, not the calc()'d size. This leads to cells that are sized far larger than their apparent content.

If you adjust percy's example code to have multiple rows, you can have multiple cells of one row expand so that they overlap the rows beneath while the row as a whole still remains the original height.  That is, all three cells in the first row can have enough text to *all* be larger than the row height, and the row height won't expand.

Basically, all of it implies that once initial values are set, the browser never percolates any further changes to the grid sizing system.

Comment 4

[Mats Palmgren (inactive)]

Attachment: testcase

(Updated the original test to account for the 'grid-template' shorthand
syntax having swapped rows/columns since this was filed.)

Comment 5

[Mats Palmgren (inactive)]

This is now fixed in the latest Nightly.  Thanks for your report.