Re: [csswg-drafts] [css-flexbox] [css-sizing-4] Interaction of flexbox minimum height and aspect-ratio minimum height (#6069)

The CSS Working Group just discussed `[css-flexbox] [css-sizing-4] Interaction of flexbox minimum height and aspect-ratio minimum height`, and agreed to the following:

* `RESOLVED: non-replaced elements with an aspect ratio honor the intrinsic size in flexbox for min-height:auto`

<details><summary>The full IRC log of that discussion</summary>
&lt;astearns> topic: [css-flexbox] [css-sizing-4] Interaction of flexbox minimum height and aspect-ratio minimum height<br>
&lt;astearns> github: https://github.com/w3c/csswg-drafts/issues/6069<br>
&lt;fremy> cbiesinger: the aspect ratio spec says that min-height auto and you compute it using the aspect ratio, you expand to the content height<br>
&lt;fremy> cbiesinger: but in a column flex box, the height is the minimum of the transferred height and the content height<br>
&lt;fremy> cbiesinger: it's not clear to me which one should win<br>
&lt;fremy> cbiesinger: fantasai clarified the aspect ratio version should win<br>
&lt;fremy> cbiesinger: but does the working group agree to this?<br>
&lt;fremy> fantasai: this is different for replaced and non-replaced elements<br>
&lt;fremy> fantasai: because the intrinsic size of replaced elements is not as strict<br>
&lt;fremy> fantasai: so we need to keep that behavior for them<br>
&lt;fremy> fantasai: but for non-replaced elements, the intrinsic size is more meaningful<br>
&lt;fremy> florian: I'm not sure about what the alternative is?<br>
&lt;fremy> cbiesinger: width:100px + aspect ratio 1:1 but intrinsic-height:200px<br>
&lt;fremy> florian: ah ok, got it<br>
&lt;fremy> cbiesinger: because it used to be the minimum<br>
&lt;fremy> jensimmo_: the default of flexbox is that it's shrink then grow<br>
&lt;fremy> jensimmo_: but the proposal is that for replaced elements, we make aspect ratio "work"<br>
&lt;fremy> fantasai: yes, that is the heart of the issue<br>
&lt;fremy> (some discussion about magic css jokes)<br>
&lt;fremy> astearns: last week there was more contention<br>
&lt;fantasai> [jen's explanation was that if there's a bunch of elements you want to be squares, unless too much content want to grow, that's the behavior we specced for block boxes with aspect-ratio; and the issue is making sure it works the same for flexbox]<br>
&lt;fremy> iank_: that was me last week<br>
&lt;fremy> iank_: I am now fine with this<br>
&lt;fremy> astearns: is the spec edit correct for what we discussed<br>
&lt;fremy> fantasai: I think it is correct<br>
&lt;fremy> fantasai: but we need to update the spec<br>
&lt;fremy> fantasai: proposal is non-replaced elements with an aspect ratio honor the intrinsic size in flexbox<br>
&lt;cbiesinger> for min-height:auto<br>
&lt;fremy> jensimmo_: I am seeing a lot of tutorials about aspect ratio on replaced elements<br>
&lt;fremy> jensimmo_: in particular content-size fit etc<br>
&lt;fremy> fantasai: that should continue to work<br>
&lt;cbiesinger> s/content-size fit/object-fit/ I think?<br>
&lt;fremy> fantasai: that is why we don't do this for replaced elements<br>
&lt;fremy> fantasai: non-replaced elements have an intrinsic size that depends on content, that might not be easy to resize<br>
&lt;fremy> jensimmo_: the previous text for images would still apply then?<br>
&lt;fremy> fantasai: yes<br>
&lt;fremy> astearns: any objection to keep fantasai's edit in the ED?<br>
&lt;fremy> RESOLVED: non-replaced elements with an aspect ratio honor the intrinsic size in flexbox for min-height:auto<br>
&lt;fremy> &lt;br length=15min /><br>
&lt;leaverou_> Partial regrets for Chris, he's having a migraine. May join later<br>
&lt;fremy> TabAtkins: might be a dwarf propaganda map, hiding their continent from the view of others then! that would be smart of them...<br>
&lt;TabAtkins> House Sivis is gnomish, not dwarvish<br>
&lt;fantasai> ScribeNick: fantasai<br>
</details>


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


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Tuesday, 6 April 2021 15:01:29 UTC