Bug 188403 - [css-grid] Update behavior of percentage row tracks and gutters
Summary: [css-grid] Update behavior of percentage row tracks and gutters
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Manuel Rego Casasnovas
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-08-07 22:29 PDT by Manuel Rego Casasnovas
Modified: 2018-08-08 02:15 PDT (History)
7 users (show)

See Also:


Attachments
Patch (120.44 KB, patch)
2018-08-07 22:35 PDT, Manuel Rego Casasnovas
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Manuel Rego Casasnovas 2018-08-07 22:29:20 PDT
This bug is to adapt WebKit to the following resolutions by the CSSWG:
* Percentage row tracks: https://github.com/w3c/csswg-drafts/issues/1921
* Percentage row gutters: https://github.com/w3c/csswg-drafts/issues/509

The change has been already done in Blink (https://chromium.googlesource.com/chromium/src/+/47bfc59ccdda82e2b28817a653c792d7a606ee8e)
so it'd be nice to follow it in WebKit too.

Change for percentage row gutters has been also done in Firefox already.
Comment 1 Manuel Rego Casasnovas 2018-08-07 22:35:14 PDT
Created attachment 346757 [details]
Patch
Comment 2 Manuel Rego Casasnovas 2018-08-07 22:48:30 PDT
Comment on attachment 346757 [details]
Patch

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

> LayoutTests/imported/w3c/web-platform-tests/css/css-grid/grid-layout-properties.html:97
> +        '<track-size>.<track-breadth>.<percentage>': ['100%', '150px 50px 50px'],

Created PR in WPT for this change (as this was not in the repo yet):
https://github.com/web-platform-tests/wpt/pull/12349

Please review and merge the PR first before landing this patch.
Comment 3 Sergio Villar Senin 2018-08-08 00:27:02 PDT
Comment on attachment 346757 [details]
Patch

Great stuff!
Comment 4 WebKit Commit Bot 2018-08-08 02:14:43 PDT
Comment on attachment 346757 [details]
Patch

Clearing flags on attachment: 346757

Committed r234687: <https://trac.webkit.org/changeset/234687>
Comment 5 WebKit Commit Bot 2018-08-08 02:14:44 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2018-08-08 02:15:38 PDT
<rdar://problem/43039111>