[webkit-reviews] review granted: [Bug 170764] [css-grid] Add support for percentage gaps : [Attachment 306897] Mini change on ChangeLog

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 18 07:43:00 PDT 2017


Sergio Villar Senin <svillar at igalia.com> has granted Manuel Rego Casasnovas
<rego at igalia.com>'s request for review:
Bug 170764: [css-grid] Add support for percentage gaps
https://bugs.webkit.org/show_bug.cgi?id=170764

Attachment 306897: Mini change on ChangeLog

https://bugs.webkit.org/attachment.cgi?id=306897&action=review




--- Comment #7 from Sergio Villar Senin <svillar at igalia.com> ---
Comment on attachment 306897
  --> https://bugs.webkit.org/attachment.cgi?id=306897
Mini change on ChangeLog

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

> LayoutTests/fast/css-grid-layout/grid-gutters-as-percentage.html:15
> +.rows50-50 { grid-template-rows: 50px 50px; }

We haven't used this kind of notation %d-%d for class names in the past. We
normally do things like two100Column or things like that.

> LayoutTests/fast/css-grid-layout/grid-gutters-get-set.html:33
> +.gridInvalidColumnGap { grid-column-gap: -webkit-max-content; }

Seems unrelated?

> LayoutTests/fast/css-grid-layout/grid-gutters-get-set.html:44
> +.gridInvalidImplicitGridGap { grid-gap: -webkit-fit-content; }

Ditto.


More information about the webkit-reviews mailing list