Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(mixins.scss): inject SassScript value to CSS variables to fix libsass 3.5.0 #1620

Merged
merged 1 commit into from
Mar 27, 2018
Merged

fix(mixins.scss): inject SassScript value to CSS variables to fix libsass 3.5.0 #1620

merged 1 commit into from
Mar 27, 2018

Conversation

VdustR
Copy link
Member

@VdustR VdustR commented Mar 23, 2018

http://sass-lang.com/documentation/file.SASS_CHANGELOG.html#3_5_0__12_July_2017_

The way CSS variables are handled has changed to better correspond to the CSS spec. They no longer allow arbitrary SassScript in their values; instead, almost all text in the property values will be passed through unchanged to CSS. The only exception is #{}, which will inject a SassScript value as before.

fix #1619

@marcosmoura marcosmoura merged commit 4f180f6 into vuematerial:dev Mar 27, 2018
@VdustR VdustR deleted the fix/scss-variable branch March 28, 2018 00:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Theme configuration broken after update to node-sass 4.8
2 participants