Re: [csswg-drafts] [css-grid] Unclear how to limit track growth by fit-content() argument (#4549)

After careful review, we actually think the spec is already complete and correct in this regard. The "Distribute extra space" algo already handles fit-content() explicitly, in exactly the way we want, in step 3. We very purposely do *not* want to apply fit-content() clamping to base sizes, *because* fit-content is supposed to respect its minimum size (here reflected as an "auto" min track sizing function) more strongly than either its specified or maximum size. All we want to do is treat the track like its max sizing function is "max-content" (thus affecting only the growth limit), *unless* that would make it larger than the specified length.

However, the line at the end of the "max-content maximums" steps in steps 2 and 3 of Resolve Intrinsic Sizes *is* misleading - as your own confusion shows, it implies something additional needs to be done, when actually it's an editorial statement referring to the fit-content() handling already done in the Distribute Extra Space algo. We'll remove those.

(That said, we've found several other minor mistakes or necessary clarifications in these sections while reviewing. :( )

GitHub Notification of comment by tabatkins
Please view or discuss this issue at using your GitHub account

Received on Wednesday, 22 July 2020 21:57:08 UTC