-
Notifications
You must be signed in to change notification settings - Fork 707
[css-display] Becoming a formatting context without display types #1563
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I suppose that becoming a formatting context implies creating a box. Maybe it's better to add a Note to the definition of these display values that they never establish a formatting context? But it's an interesting case what happens if an element with |
https://drafts.csswg.org/css-contain/#containment-layout :
In other word, "becoming a formatting context" is not invoked by the contain property on |
And the same for But the definition of "becoming a formatting context" should not rely on not being invoked with |
Sure. I am just saying that form the point of view of these specs, it doesn't matter what we define it to be. |
Boxes generate formatting contexts, not elements. I've at least patched that section of the "becoming a formatting context" definition. The question is thus moot - |
Well, before 71cfe56, it were elements what could "become a formatting context". Changing it to boxes fixes this issue, but now "becoming a formatting context" is not well-defined, because boxes don't have display types (#1480). If you plan "become a formatting context" to affect computed |
Yes, the section as a whole needs to be rewritten - it's currently marked for discussion at the F2F next month. |
Becoming a formatting context is defined as follows:
But what happens if an element that does not have inner nor outer display types (i.e. it has
display: none
ordisplay: contents
) becomes a formatting context? Such an element won't establish a formatting context, so its inner display type should be changed toflow-root
, but it doesn't have an inner display type!I think "this condition is satisfied" should also apply in that case.
The text was updated successfully, but these errors were encountered: