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-4] How to parse "contain-intrinsic-size: 10px auto 20px;" #6391
The grammar allows the length and auto in any order. Therefore, the following is valid: contain-intrinsic-size: 10px auto 20px;
However, should it be interpreted as contain-intrinsic-width: auto 10px; contain-intrinsic-height: 20px; or as contain-intrinsic-width: 10px; contain-intrinsic-height: auto 20px;?
Following up: we just fixed the ordering as auto <length>, since that reflects the importance of the values and solves the parsing issue - 10px auto 20px is thus c-i-width: 10px; c-i-height: auto 20px;.
It can be, yes, but we sometimes separate out clauses like this when the behavior is significantly different and we want to talk about the cases separately.
Activity
[css-sizing-4] Make contain-intrinsic-size shorthand parseable. #6391
tabatkins commentedon Sep 8, 2021
Following up: we just fixed the ordering as
auto <length>, since that reflects the importance of the values and solves the parsing issue -10px auto 20pxis thusc-i-width: 10px; c-i-height: auto 20px;.Loirooriol commentedon Sep 8, 2021
<length> | auto <length>could be shortened toauto? <length>tabatkins commentedon Sep 8, 2021
It can be, yes, but we sometimes separate out clauses like this when the behavior is significantly different and we want to talk about the cases separately.
cbiesinger commentedon Sep 8, 2021
Thanks, sgtm. I implemented that in https://chromium-review.googlesource.com/c/chromium/src/+/3138514.