Bug 157137

Summary: [css-grid] Unprefix CSS Grid Layout properties
Product: WebKit Reporter: Manuel Rego Casasnovas <rego>
Component: CSSAssignee: Manuel Rego Casasnovas <rego>
Status: RESOLVED FIXED    
Severity: Normal CC: 50167214, cdumez, commit-queue, darin, dino, jfernandez, simon.fraser, svillar
Priority: P2 Keywords: WebExposed
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 157134    
Bug Blocks: 60731    
Attachments:
Description Flags
Patch none

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]
Patch
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.

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

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]
Patch

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