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

Adding the data for the container* properties #657

Merged
merged 6 commits into from
Mar 30, 2023
Merged

Conversation

yarusome
Copy link
Contributor

@yarusome yarusome commented Mar 29, 2023

Description

This PR adds the data for the container, container-name, and container-type properties. It also corrects the spelling for the "listEachItemIdentifyerOrNoneAuto" key.

Motivation

Additional details

The spec links for these properties:
https://w3c.github.io/csswg-drafts/css-contain-3/#container-shorthand
https://w3c.github.io/csswg-drafts/css-contain-3/#container-name
https://w3c.github.io/csswg-drafts/css-contain-3/#container-type

CSSWG issue related to the computed value of container-type:
w3c/csswg-drafts#8643

Related issues and pull requests

Relates to mdn/content#24786.

@yarusome yarusome changed the title Container Adding the data for the container* properties Mar 29, 2023
@teoli2003
Copy link
Contributor

I fixed the schema for the typo. There is more work to be done in it and in l10n.css; I plan to do this tomorrow.

@teoli2003 teoli2003 requested a review from wbamberg March 30, 2023 05:19
Copy link
Contributor

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 thank you @yarusome and @teoli2003 . We really need a better way to do this.

@wbamberg wbamberg merged commit d0e0fce into mdn:main Mar 30, 2023
@yarusome yarusome deleted the container branch March 30, 2023 16:26
@teoli2003
Copy link
Contributor

We really need a better way to do this.

Yes, but one improvement after the other. This one is not easy.

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

Successfully merging this pull request may close these issues.

3 participants