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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add contain utilities #12993

Merged
merged 5 commits into from
Feb 21, 2024
Merged

feat: add contain utilities #12993

merged 5 commits into from
Feb 21, 2024

Conversation

KrisBraun
Copy link
Contributor

Add utilities for the contain property, which is supported in all evergreen browsers by now 馃檹馃憣

Based on #12747 by @manniL. Updated to support multiple contain keywords.

manniL and others added 3 commits February 21, 2024 13:31
Support multiple contain properties, such as "contain-size contain-layout". Drop contain-unset as we haven't added -unset variants for other utilities.
@KrisBraun KrisBraun self-assigned this Feb 21, 2024
Vite is generating files like "index--T9oO-MP.css", which required relaxing the regex used in tests.
@thecrypticace
Copy link
Contributor

Only question I have is if the contain utility is placed where we want it?

@KrisBraun
Copy link
Contributor Author

Only question I have is if the contain utility is placed where we want it?

Yes, placed after willChange as both are related to performance.

@KrisBraun KrisBraun merged commit e4154e0 into master Feb 21, 2024
10 checks passed
@KrisBraun KrisBraun deleted the manniL-feat/content-utilities branch February 21, 2024 20:55
KrisBraun added a commit that referenced this pull request Feb 23, 2024
* feat: add contain utilities

* chore: lint

* Support mulptiple contain properites

Support multiple contain properties, such as "contain-size contain-layout". Drop contain-unset as we haven't added -unset variants for other utilities.

* Update Vite; fix test regex

Vite is generating files like "index--T9oO-MP.css", which required relaxing the regex used in tests.

---------

Co-authored-by: Alexander <github@lichter.io>
KrisBraun added a commit that referenced this pull request Feb 23, 2024
* feat: add contain utilities

* chore: lint

* Support mulptiple contain properites

Support multiple contain properties, such as "contain-size contain-layout". Drop contain-unset as we haven't added -unset variants for other utilities.

* Update Vite; fix test regex

Vite is generating files like "index--T9oO-MP.css", which required relaxing the regex used in tests.

---------

Co-authored-by: Alexander <github@lichter.io>
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.

None yet

3 participants