You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[css-sizing-3] max-content sizes of replaced elements errors #6072
The conditional for when this text triggers hooks the text into the undefined cases in CSS2.1, but the text defines a bunch of cases to which the conditional doesn't apply.
The conditional is above bolded text, so it's not obvious that it limits the text below it.
The text claims to apply to boxes with / without preferred aspect ratio, but it really should apply specifically to replaced elements.
The first bullet point of the last case can't apply under the conditional, and is non-Web-compatible otherwise, so should be deleted.
Activity
[css-sizing-3] Fix the definitions of replaced max-content sizes. #6072
fantasai commentedon Mar 3, 2021
@cbiesinger @dholbert Mind reviewing the updated section? :)
cbiesinger commentedon Mar 10, 2021
In this line:
I would remove the "Note that" as it's repetitive.
Otherwise, looks good, except I have no idea if removing 300x150 from boxes with an aspect ratio is web compatible.
dholbert commentedon Mar 10, 2021
Looks good to me.
[css-sizing-3] Remove duplicated Note in Note. #6072 (comment)