search

less / less.js

Less. The dynamic stylesheet language.
http://lesscss.org
Apache License 2.0
17.02k stars 3.41k forks source link

Disabling required parentheses for mixin calls? #3584

Closed urikalish closed 3 years ago

urikalish commented 3 years ago

We're considering an upgrade (v3.13.1 --> v4.0.0), but have thousands of old-style no-parentheses mixin calls in our current code. Is it possible to disable this in some way?

matthew-dean commented 3 years ago

It's a change in parser code, so no, there's not currently a way to disable it.

Just so you know, the last, most recent v3 release and v4 are nearly identical and were released close together. v4 has the paren requirement, the parens-division default setting for math, and a few other minor things.

urikalish commented 3 years ago

I understand. Thank you for the quick reply.

matthew-dean commented 3 years ago

Maybe this change was too aggressive in the interest of syntactic conformity? I'm going to keep this open as something that would accept a PR with an option.

matthew-dean commented 3 years ago

I'm gonna roll back the parens requirement

matthew-dean commented 3 years ago

4.1.0 released with the mixin paren requirement rolled back. Hope that helps! https://github.com/less/less.js/releases/tag/v4.1.0

urikalish commented 3 years ago

Thanks!