Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[css-display] Do 'contents' and 'none' have an outer display type? #1227

Closed
Loirooriol opened this issue Apr 16, 2017 · 2 comments
Closed

[css-display] Do 'contents' and 'none' have an outer display type? #1227

Loirooriol opened this issue Apr 16, 2017 · 2 comments
Labels
Closed Accepted as Editorial css-display-3 Current Work Tracked in DoC

Comments

@Loirooriol
Copy link
Contributor

CSS Display says in Box Generation: the none and contents keywords:

Elements with either of these values do not have inner or outer display types, because they don’t generate any boxes at all.

However, in Automatic Box Type Transformations,

Some layout effects require blockification or inlinification of the box type, which sets the box’s outer display type, if it is not none or contents, to block or inline (respectively).

Which is it, do elements with display: contents or display: none have an outer display type with that value, or do they have no outer display types?
@fantasai fantasai added the css-display-3 Current Work label Apr 18, 2017
@tabatkins
Copy link
Member

Fixed up the latter sentence to call out those values as not generating a box at all, rather than having an outer display type (since outer display types are owned by boxes, which don't exist at all in this case).

@tabatkins
Copy link
Member

Fixed in f0a2794

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Closed Accepted as Editorial css-display-3 Current Work Tracked in DoC
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tabatkins @fantasai @Loirooriol