W3C home > Mailing lists > Public > www-style@w3.org > May 2014

RE: [css3-flexbox] Bad result in flex algorithm when combining stretch, and elements with an aspect ratio

From: Rossen Atanassov <Rossen.Atanassov@microsoft.com>
Date: Wed, 7 May 2014 16:29:49 +0000
To: fantasai <fantasai.lists@inkedblade.net>, "Kang-Hao (Kenny) Lu" <kanghaol@oupeng.com>
CC: Tab Atkins Jr. <jackalmage@gmail.com>, www-style list <www-style@w3.org>
Message-ID: <515dd040ff3f4dabbf9b4be95b22b40c@BN1PR03MB187.namprd03.prod.outlook.com>
> -----Original Message-----
> From: fantasai [mailto:fantasai.lists@inkedblade.net]
> Sent: Monday, February 4, 2013 10:42 PM
> 
> You're right. We added that the container's inner cross size should be
> clamped to the item's min/max cross size before it's used to compute the
> flex base size.
> 
>    # If the flex basis and cross size are both ‘auto’ , the flex
>    # container is single-line and has a definite cross size, the flex
>    # item has an intrinsic aspect ratio, and the flex item has
>    # ‘align-self: stretch’, the flex base size is computed from the
>    # flex container's inner cross size (clamped to the flex item's
>    # min and max cross size) and the flex item's intrinsic aspect
>    # ratio.
> 
> Please let us know if this looks correct yet. :)

In review of the current draft it looks like this thread was used to address issue #8 [1]. If this is the case then I don't believe we've solved the issue raised here. Here's the current definition:

------------
If the flex item has ... 
 an intrinsic aspect ratio, 
 a flex basis of ‘auto’, and 
 a definite cross size <<< problem
------------

The problem is that we allow the intrinsic size to kick in only when the cross size if defined and not in the case it is 'auto' and 'stretch'.

The fix should be as simple as adding the OR statement for 'auto' and 'stretch'. 

If you're OK with that I'll go ahead and add the change.

Thanks,
Rossen


[1] http://dev.w3.org/csswg/css-flexbox-1/issues-cr-2012#issue-8 
Received on Wednesday, 7 May 2014 16:30:30 UTC

This archive was generated by hypermail 2.3.1 : Monday, 2 May 2016 14:39:21 UTC