[go: nahoru, domu]

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.

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

[css-contain] container-type shouldn't allow mixing normal with other values #7669

Closed
Loirooriol opened this issue Aug 31, 2022 · 3 comments
Closed

Comments

@Loirooriol
Copy link
Contributor

https://drafts.csswg.org/css-contain-3/#container-type

Value: normal || [ size | inline-size ]

But normal size and normal inline-size don't make sense.

normal establishes a query container for container style queries, but that's the default, included in size and inline-size:

Unless otherwise noted, all elements are query containers for the purpose container queries that do no require explicit containment (such as container style queries)

The definitions are even contradictory, e.g. with normal size, both of these apply:

The element is not a query container for any container size queries

Establishes a query container for container size queries

So the syntax should just be normal | size | inline-size.

@Loirooriol
Copy link
Contributor Author

@mirisuzanne Seems a mistake from 3aed70b, it was

none | [ style || [ size | inline-size ] ]

and then style was removed (included in the others), so it should be

none | [ size | inline-size ]

@mirisuzanne
Copy link
Contributor

@Loirooriol should be fixed in 964c07f.

@mirisuzanne
Copy link
Contributor

(It looks to me like the WP Tests are accurate already)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants