Skip to content

Update "contain" property to accept "inline-size"#372

Closed
Seirdy wants to merge 1 commit intow3c:mainfrom
Seirdy:feat/contain-inline-size
Closed

Update "contain" property to accept "inline-size"#372
Seirdy wants to merge 1 commit intow3c:mainfrom
Seirdy:feat/contain-inline-size

Conversation

@Seirdy
Copy link
Copy Markdown
Contributor

@Seirdy Seirdy commented Aug 5, 2022

Closes #370

@ylafon
Copy link
Copy Markdown
Member

ylafon commented Aug 7, 2022

Do you have a pointer to the specification? It is useful as a reference (see at the top of the file), and also to ensure that the patch is the right one.
Thanks!

@Seirdy
Copy link
Copy Markdown
Contributor Author

Seirdy commented Aug 7, 2022

@Seirdy Seirdy force-pushed the feat/contain-inline-size branch from 21bf947 to 22c01f0 Compare August 7, 2022 20:03
@Seirdy
Copy link
Copy Markdown
Contributor Author

Seirdy commented Aug 7, 2022 via email

@ylafon
Copy link
Copy Markdown
Member

ylafon commented Aug 22, 2022

Looking at https://www.w3.org/TR/2022/WD-css-contain-3-20220818/#contain-property it seems that another check is required.
Note that the grammar changed, and this is the reason we try to implement only what is somehow stable.

ylafon added a commit that referenced this pull request Apr 9, 2024
@ylafon
Copy link
Copy Markdown
Member

ylafon commented Apr 9, 2024

Closed per 78ebb17

@ylafon ylafon closed this Apr 9, 2024
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.

Support contain: inline-size property.

2 participants