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

Replace deprecated SASS methods #2100

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

aleho
Copy link

@aleho aleho commented Sep 23, 2024

Recently, compiling with SASS 1.79, some color functions now output deprecation warnings.

This PR keeps the legacy behavior as documented at e.g., https://sass-lang.com/documentation/modules/color/#darken.

Copy link

@javierjulio javierjulio left a comment

Choose a reason for hiding this comment

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

@aleho thank you for addressing the deprecations and submitting this. Could you please update the settings file to resolve the other deprecations there?

@joeworkman if this will get updated, would you accept this change and cut a release? It would help to further reduce the Dart Sass deprecations.

scss/settings/_settings.scss Show resolved Hide resolved
*) map-get
*) floor, percentage
*) lighten, darken, scale-color
Copy link

@javierjulio javierjulio left a comment

Choose a reason for hiding this comment

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

Thanks!

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.

2 participants