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

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.)

~Daniel

Received on Tuesday, 8 July 2014 23:58:01 UTC