Document use of eval() to parse calculations

[traviskaufman]

I noticed that the way math expressions are reduced is by using eval() (

reduce-css-calc/index.js

Line 75 in 1ce8bd0

result = eval(toEvaluate)

), however this isn't documented anywhere. Technically, the w3c spec calls for calc to only accept simple binary operators (https://www.w3.org/TR/css3-values/#calc-notation), but with this you could also do things like

.foo {
  width: calc(2.5px >>> 0);
}

See this codepen as an example.

Personally I view this as a feature, and feel like it should be documented rather than just being an implementation detail. What do you think?

[MoOx]

I think this is not documented because this is not compatible with specs (and this plugins aims to reduce css calc, following specs). So I don't think it's a good idea. Ideally, eval() should not be used, but I was lazy when I made this.

[ChALkeR]

@MoOx Is this still applicable? There is no eval() anymore. 😉

[MoOx]

My guess it that the package used still offers more than CSS specs...

[ben-eb]

The original example no longer works, as eval has been removed from the module. I think it's safe to close this issue, as the intention behind the module is to work with standard syntax. 😄