Re: [css-flexbox] Possible typo w/ "a flex basis of 'main-size'" in section on getting flex basis from aspect-ratio

On Tue, Jul 8, 2014 at 4:57 PM, Daniel Holbert <dholbert@mozilla.com> wrote:
> Hi Tab,
>
> The flexbox ED currently has:
>   # 3.  Determine the flex base size and
>   #     hypothetical main size of each item:
>   #   [...snip...]
>   #   B. If the flex item has ...
>   #       * an intrinsic aspect ratio,
>   #       * a flex basis of 'main-size', and
>   #       * a definite cross size
>   #      then the flex base size is calculated
>   #      from its inner cross size and the flex
>   #      item's intrinsic aspect ratio.
> http://dev.w3.org/csswg/css-flexbox/#algo-main-item
>
> I think that second bullet-point needs s/'main-size'/'auto'/.
>
> Otherwise, this would only allow flex items with 'main-size'-dependent
> flex basis values to be sized according to their intrinsic ratio. And
> that seems like an odd and arbitrary restriction.
>
>
> In particular, if we had ...
>
>  <div style="display:flex; flex-direction: column">
>    <img style="width:10px; flex-basis:auto">
>  </div>
>
> ...then I'd expect that we *should* use the image's intrinsic ratio and
> definite cross size (10px) to resolve its "flex-basis:auto".  (but the
> spec currently would tell us not to do that, unless we instead had
> "flex-basis:main-size; height:auto")
>
> (I'm guessing this text was just converted to use 'main-size'
> accidentally as part of the recent renaming; but in this case, I think
> 'auto' is still really what we want to have here.)

Yup, fixed.

~TJ

Received on Monday, 18 August 2014 23:40:34 UTC