Skip to content

Fix use of deprecated sass lighten() and darken()#1548

Open
mattxwang wants to merge 2 commits intomainfrom
fix-darken-warnings
Open

Fix use of deprecated sass lighten() and darken()#1548
mattxwang wants to merge 2 commits intomainfrom
fix-darken-warnings

Conversation

@mattxwang
Copy link
Member

@mattxwang mattxwang commented Oct 9, 2024

Sass has deprecated the lighten() and darken() functions. color.adjust is a drop-in replacement, but does require importing sass:color. This is a problem, since this module only (?) exists in dart-sass, which earlier versions of the Jekyll gem (most notably: pages-gem users) will not have access to.

Closes #1541.

@emiltin
Copy link

emiltin commented Oct 9, 2024

This would be nice to fix :-)

@mattxwang
Copy link
Member Author

For @emiltin (and others) - copying over my comment from the issue (as it should help you resolve in the mean time):


Ah - okay, slightly bad news. Users of the github-pages gem may not have access to the color module, since they don't use dart-sass. Here's the upstream issue: jekyll/jekyll#9686

I will have to think about a way to resolve this issue in the general case. I think it's unlikely I will push a fix very quickly (especially as my bandwidth has been awful recently). My suggestion is to either:

  • pin your version of sass-embedded to a version that doesn't have this warning (e.g. 1.78.0 downwards)
  • add to your _config.yml
sass:
  quiet_deps: true

Sorry for the confusion! Unfortunately, a general solution is slightly out of my hands unless we drop support for pages-gem. I'll keep this issue and the associated PR #1548 with my findings. Thanks!

@mattxwang mattxwang added the status: blocked Issues that are blocked for an external reason. label Oct 16, 2024
usta added a commit to endoflife-date/endoflife.date that referenced this pull request Jan 7, 2025
usta added a commit to endoflife-date/endoflife.date that referenced this pull request Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: blocked Issues that are blocked for an external reason.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecation warnings about SASS darken()

2 participants