Conversation
✅ Deploy Preview for sass-lang ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
|
||
| However, due to an oversight in the implementation of both Ruby Sass and Dart | ||
| Sass, this was never enforced. It was possible to write a function call like | ||
| `rgb([1, 2]..., 3)` with the rest argument before a positional (or named) |
There was a problem hiding this comment.
Just out of curiosity I tried passing 2 lists with ... and I do get an error, but it's not very descriptive.
Plain CSS functions don't support keyword arguments.
╷
4 │ b: g([1, 2]..., [3, 4]...)
Maybe this is fine, but I just wanted to let you know. This is definitely outside of the scope of the current fix anyways
There was a problem hiding this comment.
Two lists is intentionally supported, so that you can write something like rgb([1, 2]..., (blue: 3)...). If you want to take a crack at making the error message more descriptive if the second argument isn't a map, go for it.
There was a problem hiding this comment.
@Goodwine the error in your code snippet is because you used g() instead of the name of a Sass function. The error you get is unrelated to the current deprecation (it is related to the restriction of calls for CSS functions).
3 participants
See sass/sass#4120
See #1446