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

Spam #2302

Closed
ghost opened this issue Mar 22, 2018 · 0 comments
Closed

Spam #2302

ghost opened this issue Mar 22, 2018 · 0 comments
Labels

Comments

@ghost
Copy link

ghost commented Mar 22, 2018

  • NPM version (npm -v):
  • Node version (node -v):
  • Node Process (node -p process.versions):
  • Node Platform (node -p process.platform):
  • Node architecture (node -p process.arch):
  • node-sass version (node -p "require('node-sass').info"):
  • npm node-sass versions (npm ls node-sass):
@xzyfer xzyfer closed this as completed Mar 22, 2018
@nschonni nschonni changed the title e Spam Apr 3, 2018
jiongle1 pushed a commit to scantist-ossops-m2/node-sass that referenced this issue Apr 7, 2024
…ass#2361)

* Make the deprecation function more generic

* Produce a deprecation warning for ID strings that look like colors

Fixes sass#2302
Spec sass/sass-spec#1099
jiongle1 pushed a commit to scantist-ossops-m2/node-sass that referenced this issue Apr 7, 2024
A deprecation for the old behaviour landed in sass#2302. This PR removes
the deprecation warning for the old behaviour, and starts parsing
4 and 8 digit hex values as Colors.

Looks like we had support for 8 digit hex colors for a while but
somehow 4 digit support was missed.

Spec sass/sass-spec#1273

Fixes sass#2674
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants