[css-sizing] Intrinsic size of replaced elements incorrect #794

Open
fantasai opened this Issue Dec 7, 2016 · 3 comments

Projects

None yet

3 participants

@fantasai
Contributor
fantasai commented Dec 7, 2016

Currently Sizing 3 defines intrinsic sizes of replaced elements explicitly.
This definition isn't correct, and it should be deleted in favor of falling through to the CSS2.1 reference used for non-replaced elements.

@fantasai fantasai added a commit that referenced this issue Dec 19, 2016
@fantasai fantasai [css-sizing] Defer sizing of replaced elements to CSS2.1 algorithms, …
…since the one here isn't quite correct. Addresses #794.
f0d83c2
@fantasai fantasai added the Agenda+ label Dec 19, 2016
@fantasai
Contributor

Agenda+ to point out that the implication here is that, when an aspect ratio is present, min-content and max-content are dependent on sizing constraints in the opposite axis. (This might be surprising/disturbing to implementers but is probably as expected for authors.)

@dbaron
Contributor
dbaron commented Dec 20, 2016

I really don't want min-content and max-content to be anything other than intrinsic sizes. And, in particular, I want them never to depend on the element's own specified sizes, precisely so that they can be used as keywords for the element's specified sizes.

Doing so could cause problems down the road if we ever add the ability for calc() expressions to reference concepts like "this element's min-content size in the other dimension" or "the grandparent element's size". (The latter is harder than the former, but we might be able to come up with a set of ways where it does make sense or a way of resolving the constraints, and having things like "intrinsic" sizes that are almost but not quite always intrinsic would make that harder.)

@fantasai
Contributor
fantasai commented Jan 4, 2017 edited

The problem with that is then we don't have a keyword for "shrinkwrap this thing as if it was a float with (infinite/minimal space)", which is what you generally want in e.g. grid track sizing.

@fantasai fantasai added Agenda+ F2F and removed Agenda+ labels Jan 4, 2017
@astearns astearns removed the Agenda+ F2F label Jan 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment