New issue
Advanced search Search tips

Issue 750726 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[css-grid] Intrinsic track sizes are wrong calculated for items with vertical percentage paddings/margins

Project Member Reported by dholb...@gmail.com, Jul 31 2017

Issue description

Chrome Version:Version 61.0.3163.13 (Official Build) dev (64-bit)
OS: Ubuntu 16.04 (Linux)

What steps will reproduce the problem?
(1) Load attached testcase, or jsfiddle: https://jsfiddle.net/j9rL00ar/

What is the expected result?
 The orange area (grid item) should not overflow from the black border (grid).  The grid has height:auto, so it should be tall enough to contain the intrinsic height of its child.

What happens instead?
 Orange area overflows black border. (Grid doesn't shrinkwrap its item's height.)

Please use labels and text to provide additional information.


For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.

 
chrome-bug.html
237 bytes View Download

Comment 1 by e...@chromium.org, Aug 1 2017

Cc: -cbiesinger@google.com -ojan@chromium.org svil...@igalia.com
Status: Available (was: Untriaged)

Comment 2 by r...@igalia.com, Aug 1 2017

Cc: jfernan...@igalia.com r...@igalia.com
Summary: [css-grid] Intrinsic track sizes are wrong calculated for items with vertical percentage paddings/margins (was: Grid items overflow from the grid, if they have vertical percent padding (i.e. grid auto-height is miscalculated))
Yeah the issue happens with vertical percentage paddings and margins.
The problem is that the track size is wrongly computed sometimes
(it seems the value change depending on if we've repeated layouts or not).
The size of the grid container is the same than the only track,
so that's working fine, but the track is the one that is wrong.

I think that the issue is that the percentage is first resolved against
the intrinsic size of the item, and that's the value used for the "auto" track.
But later when the percentage is actually resolved, the "auto" track size
is not updated.

Comment 3 by r...@igalia.com, Apr 26 2018

Owner: r...@igalia.com
Status: WontFix (was: Available)
This seems to be fixed for a while, I see it working properly in M66 already.

I guess it was fixed by other patches related to percentages and paddings/margins.

Sign in to add a comment