Re: [csswg-drafts] [css-flexbox] Change content-size suggestion to min-intrinsic instead of min-content? (#6794)

@tabatkins , one thing I'm confused about in [your analysis](https://github.com/w3c/csswg-drafts/issues/6794#issuecomment-1026183636) -- you say:
> So the flex base size is 100px (from the transferred width), and the automatic minimum size is zero

Why do you say the automatic minimum size is zero -- shouldn't it actually be 100px?

The flex item here should use the content-size suggestion i.e. its min-content height, per https://drafts.csswg.org/css-flexbox-1/#min-size-auto (since it doesn't have a specified size suggestion and is not replaced).

And I think its min-content height is 100px, not 0; at least, [this example (with explicit `height:min-content`)](https://livedomviewer.com/?%3C!DOCTYPE%20html%3E%0A%3Cdiv%20style%3D%22aspect-ratio%3A%201%2F1%3B%0A%20%20%20%20%20%20%20%20%20%20%20%20width%3A%20100px%3B%20height%3A%20min-content%3B%0A%20%20%20%20%20%20%20%20%20%20%20%20background%3A%20blue%3B%20border%3A%205px%20solid%20orange%22%3E%0A%3C%2Fdiv%3E) suggests that it is:

```html
<!DOCTYPE html>
<div style="aspect-ratio: 1/1;
            width: 100px; height: min-content;
            background: blue; border: 5px solid orange">
</div>
```

Browsers agree that this^ div is 100px tall, which I think (?) means its `height:min-content` is resolving to 100px and hence that's its min-content height? So I'd think that that's what the content size suggestion should be, when resolving its automatic minimum size in Tab's analysis above.

-- 
GitHub Notification of comment by dholbert
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/6794#issuecomment-1027455922 using your GitHub account


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

Received on Wednesday, 2 February 2022 00:41:49 UTC