borodean/less-properties

[Deprecated] Mixin that adds some variable property support to LESS.

less-properties's Issues

add a license

Have you considered adding a license?

Maybe MIT or Apache? or perhaps the WTFPL if you're into that sorta thing.

Just please not GPL, LGPL or BSD :)

Thanks!

Allow passing a value with spaces without wrapping it as an escaped string

With any value that has spaces in it (e.g. 1px solid red), fails to generate a proper CSS definition since LESS treats the argument as an array of values. You can get around this in your solution by passing it in as an escaped string (e.g. ~"1px solid red"). Out of your test cases, the following seem to contain this bug:

.test-compound
.test-function-param-preserve-second
.test-function-param-preserve-first
.test-function-param-preserve-double
.test-nested-coumpoun

Here is an example of the failing output:

.property(transition, all .5s ease, "webkit moz o");

.test-compound {
  -test-should: generate a property definition that looks correct;
  -less-property: ;
  -webkit-transition: [all, 0.5s, ease];
  -moz-transition: [all, 0.5s, ease];
  -o-transition: [all, 0.5s, ease];
  transition: [all, 0.5s, ease];
}

In my solution, I encountered this same problem, but resolved it by escaping the argument before using it. In your mixin, if you escape both the @property and @value arguments into new variables, I think it would be more robust. For example:

.property(@property, @value, @prefixes: '', @base: true) {
  @prop: e('@{property}');
  @vals: e('{@value}');

  // ...
}