Re: [csswg-drafts] [css-grid-1] Define computed value of grid-template-rows/columns

The CSS Working Group just discussed `Define computed value of grid-template-rows/columns`, and agreed to the following:

* `RESOLVED: Accept this edit: https://github.com/w3c/csswg-drafts/commit/63a1d65afa5997a56b210c1cbddccdb2603ecec9`

<details><summary>The full IRC log of that discussion</summary>
&lt;dael> Topic: Define computed value of grid-template-rows/columns<br>
&lt;dael> github: https://github.com/w3c/csswg-drafts/issues/3154#issuecomment-428002686<br>
&lt;dael> fantasai: As part of fix computed values I noticed this. Wanted to cross check witht he group this seemed reasonable<br>
&lt;dael> fantasai: Current ED says [reads]<br>
&lt;astearns> https://github.com/w3c/csswg-drafts/commit/63a1d65afa5997a56b210c1cbddccdb2603ecec9<br>
&lt;dael> fantasai: What this means is no distinction in computed value of 50px and minmax(50px)50px. functionally same so shouldn't be a distinction<br>
&lt;fantasai> minmax(50px,50px)<br>
&lt;dael> astearns: Any idea about what's impl?<br>
&lt;dael> fantasai: Didn't check<br>
&lt;dael> fantasai: It's hard to check because gCS is the used value for these prop so can't retrive computed value<br>
&lt;dael> fantasai: Main check would be animating between. Not sure animation is well supported<br>
&lt;dael> fantasai: This would clarify<br>
&lt;dael> rego: I think animations aren't working on any mpl yet so hard to check<br>
&lt;dael> astearns: I like the clarification and seems reasonable to me. Haven't thought through implications for animations<br>
&lt;dael> fantasai: What this does is makes it as likely as possible to animate between 2 values so that functionally same values compute to each other. Everything matchs so values that mean the same are represented same.<br>
&lt;dael> fantasai: Additional magic we might want for animating, but that's issue #3201. Computed value that's best we can do to make animation as easy as can be<br>
&lt;dael> astearns: Other comments?<br>
&lt;dael> astearns: Concerns?<br>
&lt;dael> emilio: Can test be written? All this is tricky and to get interop tests would be best<br>
&lt;dael> fantasai: Tests would be for animation and this is not being animated as far as I know. Be we can write tests<br>
&lt;dael> astearns: When someone impl animations for these properties tests will need to be written<br>
&lt;emilio> yeah, fair enough<br>
&lt;dael> astearns: Concern or objection on taking this edit: https://github.com/w3c/csswg-drafts/commit/63a1d65afa5997a56b210c1cbddccdb2603ecec9 ?<br>
&lt;dael> RESOLVED: Accept this edit: https://github.com/w3c/csswg-drafts/commit/63a1d65afa5997a56b210c1cbddccdb2603ecec9<br>
&lt;fantasai> https://github.com/w3c/csswg-drafts/issues/3201<br>
&lt;dael> fantasai: Issue #3201 is looking for impl feedback<br>
</details>


-- 
GitHub Notification of comment by css-meeting-bot
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/3154#issuecomment-430695483 using your GitHub account

Received on Wednesday, 17 October 2018 16:23:37 UTC