Skip to content

Conversation

@crisbeto
Copy link
Member

Sass has deprecated the if function in order to support the native CSS if(). These changes update all of our usages so users don't get deprecation warnings.

Note that I decided to replace it with @if/@else, instead of the new syntax, because the new syntax is:

  1. Harder to follow, in my opinion.
  2. Likely going to break users that aren't on the absolute latest version of Sass.

@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Jan 19, 2026
@pullapprove pullapprove bot requested a review from devversion January 19, 2026 11:44
[Sass has deprecated the `if` function](https://sass-lang.com/documentation/breaking-changes/if-function) in order to support the native CSS `if()`. These changes update all of our usages so users don't get deprecation warnings.

Note that I decided to replace it with `@if`/`@else`, instead of the new syntax, because the new syntax is:
1. Harder to follow, in my opinion.
2. Likely going to break users that aren't on the absolute latest version of Sass.
@crisbeto crisbeto force-pushed the sass-fn-deprecation branch from 5027dab to e0c8af2 Compare January 19, 2026 14:16
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants