Bug 182922 – [css-grid] Incorrect calculation of repeat() with multiple values, auto-fill and grid-gap

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Roman Komarov]

Created attachment 334146 [details]
Screenshot of the bug

Steps to reproduce the problem:

1. Go to https://codepen.io/kizu/pen/paayMm?editors=1100
2. Resize the viewport
3. Look at second example

What is the expected behavior?
The second example (the one with the bigger grid-gap) should work as well as the first one

What went wrong?
For some viewport values the width of the second example is calculated incorrectly, so there is a scrollbar created for the page and the layout looks broken.

Did this work before? No 

Does this work in other browsers? Yes

Chrome version: 62.0.3202.94  Channel: n/a
OS Version: OS X 10.13.2
Flash Version: 

The bigger the grid-gap, the bigger the problem, the first example at the codepen is also buggy, but as the grid-gap is very small, its almost unnoticable.

Other browsers: Edge works properly in all cases, but Firefox doesn't support multiple values at repeat() with auto-fill at all — https://bugzilla.mozilla.org/show_bug.cgi?id=1341507

[Carlos Alberto Lopez Perez]

This very same issue was reported for Chromium at https://crbug.com/813511 (already fixed)

[Carlos Alberto Lopez Perez]

A test case for this its available on WPT: http://wpt.live/css/css-grid/grid-definition/grid-auto-repeat-multiple-values-001.html

[Oriol Brufau]

Created attachment 399861 [details]
Patch

[Manuel Rego Casasnovas]

Comment on attachment 399861 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=399861&action=review

r=me

> Source/WebCore/ChangeLog:15
> +        This patch modifies two places that relates to the gaps between the

Nit: s/relates/relate/

[Oriol Brufau]

Created attachment 399878 [details]
Patch

[EWS]

Committed r261949: <https://trac.webkit.org/changeset/261949>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 399878 [details].

[Radar WebKit Bug Importer]

<rdar://problem/63461028>