Re: [css-grid] Update minmax() description now that fr is invalid as minimum

On 13/05/16 18:52, Manuel Rego Casasnovas wrote:
> 
> 
> On 12/05/16 17:52, Manuel Rego Casasnovas wrote:
>> a recent change [1] in the spec makes flex sizes invalid
>> on the min slot of minmax():
>> "  <track-size> =
>>      <track-breadth> |
>>      minmax( <inflexible-breadth> , <track-breadth> )"
>>
>> However, the text defining minmax() hasn't been updated [2]:
>> "As a maximum, a <flex> value sets the track’s flex factor.
>>  As a minimum, it is treated as zero (or min-content,
>>  if the grid container is sized under a min-content constraint)."
>>
>> I believe we should change this to state that as a minimum it makes the
>> declaration invalid.
> 
> And not only that, also the text in the algorithm needs to be updated
> too [3]:
> "When the grid container is being sized under a min-content constraint,
>  a track with a flexible min track sizing function is treated as if
>  its min track sizing function was min-content for the purposes
>  of this step."
> 
> But now a "track with a flexible min track sizing function" is invalid.

And another issue when the algorithm terms are defined [4]:
"min track sizing function
   If the track was sized with a minmax() function,
   this is the first argument to that function.
   Otherwise, it’s the track’s sizing function."

Now that "fr" is invalid as minimum,
I guess we need an exception in this rule too.

Bye,
  Rego

>> [1]
>> https://github.com/w3c/csswg-drafts/commit/4040ba7fa068905f7fe1c1bf35d665a2062acafa
>> [2] https://drafts.csswg.org/css-grid/#valdef-grid-template-columns-minmax
> [3] https://drafts.csswg.org/css-grid/#algo-content
[4] https://drafts.csswg.org/css-grid/#algo-terms

Received on Monday, 16 May 2016 14:52:44 UTC