Skip to content

Support new CSS if()#472

Merged
aeschli merged 8 commits intomicrosoft:mainfrom
ej-shafran:support-new-css-if
Feb 23, 2026
Merged

Support new CSS if()#472
aeschli merged 8 commits intomicrosoft:mainfrom
ej-shafran:support-new-css-if

Conversation

@ej-shafran
Copy link
Contributor

Spec

Sources in the spec:

Notes

This is my first time opening a PR here, but I was trying to solve SCSS deprecation issues in one of my projects and ran into this issue, so I was motivated to fix this. I hope I got the gist of how to properly parse things, but if I returned null where I was supposed to markError, or if I checked isErroneous instead of returning null, etc. - I'd be glad to fix any mistakes.

Relevant Issues

Closes #466.

The spec mentions that @container can also be defined using boolean-expression, so I can rewrite the parsing for container queries to reuse _parseBooleanExpression - and that will let me easily solve #454 as well.

@ej-shafran
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Hilma"

@aeschli aeschli enabled auto-merge (squash) February 23, 2026 14:20
@aeschli
Copy link
Collaborator

aeschli commented Feb 23, 2026

Thanks @ej-shafran !

@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 23, 2026
@aeschli aeschli merged commit 2a8cf1f into microsoft:main Feb 23, 2026
3 checks passed
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.

New CSS if() not yet supported

3 participants