Skip to content

Tweak Sass styles#832

Merged
mixonic merged 1 commit into3.0-betafrom
twokul/interpolation-warnings
Aug 7, 2020
Merged

Tweak Sass styles#832
mixonic merged 1 commit into3.0-betafrom
twokul/interpolation-warnings

Conversation

@twokul
Copy link
Contributor

@twokul twokul commented Aug 7, 2020

After we have updated ember-cli-sass to 10, the following warnings
started popping up:

WARNING: You probably don't mean to use the color value silver in interpolation here.
It may end up represented as silver, which will likely produce invalid CSS.
Always quote color names when using them as strings or map keys (for example, "silver").
If you really want to use the color value here, use '"" + $color-name'.

This change will get rid of the warnings without changing the output
CSS.

After we have updated `ember-cli-sass` to `10`, the following warnings
started popping up:

```sh
WARNING: You probably don't mean to use the color value silver in interpolation here.
It may end up represented as silver, which will likely produce invalid CSS.
Always quote color names when using them as strings or map keys (for example, "silver").
If you really want to use the color value here, use '"" + $color-name'.
```

This change will get rid of the warnings without changing the output
CSS.
@twokul twokul requested a review from mixonic August 7, 2020 16:42
@mixonic mixonic merged commit 25c4e30 into 3.0-beta Aug 7, 2020
@mixonic mixonic deleted the twokul/interpolation-warnings branch August 7, 2020 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants