[css-grid]min-content/max-content prefix in Safari

[yisibl]

input:

.foo {
     grid-template-columns: minmax(100px, min-content);
}

output:

.foo {
     -ms-grid-columns: minmax(100px, min-content);
         grid-template-columns: minmax(100px, min-content);
}

These properties just remove the prefix in https://trac.webkit.org/changeset/213831 , but I'm not sure which version of Safari.

@mrego Can you help me?

http://caniuse.com/#feat=css-grid

[mrego]

Apple doesn't publicly talk about their Safari releases, so we don't know.
I don't think it would be part of 10.1 as the beta has been around for a while, so probably Safari 11 or 10.2 if that ever exists.
Sorry for not being really helpful, you'd need to wait and check when it's or not included.

[ai]

@yisibl what is expected output?

[yisibl]

@mrego Well, it looks like caniuse's data is inaccurate,in safari 10.1, these values must be prefixed, right?

@ai expected output:

.foo {
    -ms-grid-columns: minmax(100px, min-content);
    grid-template-columns: minmax(100px, -webkit-min-content); /* Safari 10.1 or 10.2 */
    grid-template-columns: minmax(100px, min-content);
}

[ai]

@yisibl ouh, I will add grid properties to min-content https://github.com/postcss/autoprefixer/blob/master/data/prefixes.coffee#L292

Do you want to make PR to save your name in history (again)? :)

[mrego]

One special thing here, fit-content cannot be prefixed inside grid properties, or it won't work. I know it's a weird situation but that's how things are right now.
So valid declarations would be:
grid-template-columns: -webkit-min-content -webkit-max-content fit-content(500px);
However the following one would be invalid:
grid-template-columns: -webkit-min-content -webkit-max-content -webkit-fit-content(500px);

Note that fit-content in the grid properties is not a keyword like min|max-content but a kind of "function". That's the reason why it was implemented as something new and unprefixed since the beginning.

[yisibl]

@mrego OK, I see, fit-content() no prefix.

@ai Yeah, willing to serve you 👻 .

[yisibl]

Safari 10.1 shipped: https://developer.apple.com/library/prerelease/content/releasenotes/General/WhatsNewInSafari/Articles/Safari_10_1.html#//apple_ref/doc/uid/TP40014305-CH12-DontLinkElementID_55

[yisibl]

Add test case: #805

[ai]

Issue with test pull request — ideal issue 👍

Thanks, it will help me a lot.

[ai]

Done 99def37