Bug 157137 - [css-grid] Unprefix CSS Grid Layout properties
Summary: [css-grid] Unprefix CSS Grid Layout properties
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Manuel Rego Casasnovas
Keywords: WebExposed
Depends on: 157134
Blocks: 60731
  Show dependency treegraph
Reported: 2016-04-28 07:29 PDT by Manuel Rego Casasnovas
Modified: 2016-05-24 00:38 PDT (History)
8 users (show)

See Also:

Patch (1.98 MB, patch)
2016-04-29 00:27 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 2016-04-28 07:29:41 PDT
After adding the runtime feature in bug #157134 we should be able to unprefix the CSS Grid Layout properties.

More info at: https://lists.webkit.org/pipermail/webkit-dev/2016-April/028192.html
Comment 1 Manuel Rego Casasnovas 2016-04-29 00:27:22 PDT
Created attachment 277680 [details]
Comment 2 Manuel Rego Casasnovas 2016-05-04 08:16:44 PDT
It seems that after the discussion on the mailing list everybody is fine with unprefixing grid layout.

So I'm pinging reviewers, please could you take a look?
Otherwise I'll need to upload a new version as soon as some new grid layout patches land.

Comment 3 Simon Fraser (smfr) 2016-05-05 12:44:27 PDT
Comment on attachment 277680 [details]

r=me if you're ok removing support for the prefixed version.
Comment 4 Sergio Villar Senin 2016-05-06 00:24:49 PDT
(In reply to comment #3)
> Comment on attachment 277680 [details]
> Patch
> r=me if you're ok removing support for the prefixed version.

r=me too

I think we're safe removing it. We haven't shipped the feature, it has only been enabled in the last Safari technology preview and both Firefox and Chrome are using unprefixed versions. Last but not least, as Manuel says, almost all the examples out there are also using unprefixed versions.
Comment 5 WebKit Commit Bot 2016-05-06 01:16:59 PDT
Comment on attachment 277680 [details]

Clearing flags on attachment: 277680

Committed r200510: <http://trac.webkit.org/changeset/200510>
Comment 6 WebKit Commit Bot 2016-05-06 01:17:04 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 yisibl 2016-05-24 00:38:08 PDT
missing -webkit-min-content and -webkit-max-content