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.

*) 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!

@aleho
Copy link
Author

aleho commented Feb 10, 2025

@javierjulio Is there anything else to do on this PR?

@javierjulio
Copy link

@aleho no, not that I know of. Sorry, I'm not a maintainer. This depends on @joeworkman approving and merging. There is another set of PRs for foundation-sites to resolve SASS deprecations that are all ready for review.

@aleho
Copy link
Author

aleho commented Feb 11, 2025

@javierjulio Sorry, my bad! I assumed because you reviewed it you'd be able to merge. Thank you anyway!

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