Fixes #3586 (#3589)

less · Jan 29, 2021 · f727b26 · f727b26
1 parent 63c1827
commit f727b26
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/packages/less/src/less/tree/dimension.js b/packages/less/src/less/tree/dimension.js
@@ -66,7 +66,8 @@ Dimension.prototype = Object.assign(new Node(), {
     // so `1px + 2` will yield `3px`.
     operate(context, op, other) {
         /* jshint noempty:false */
-        let value = this._operate(context, op, this.value, other.value), unit = this.unit.clone();
+        let value = this._operate(context, op, this.value, other.value);
+        let unit = this.unit.clone();
 
         if (op === '+' || op === '-') {
             if (unit.numerator.length === 0 && unit.denominator.length === 0) {

diff --git a/packages/less/src/less/tree/operation.js b/packages/less/src/less/tree/operation.js
@@ -29,8 +29,11 @@ Operation.prototype = Object.assign(new Node(), {
             if (b instanceof Dimension && a instanceof Color) {
                 b = b.toColor();
             }
-            if (!a.operate) {
-                if (a instanceof Operation && a.op === '/' && context.math === MATH.PARENS_DIVISION) {
+            if (!a.operate || !b.operate) {
+                if (
+                    (a instanceof Operation || b instanceof Operation)
+                    && a.op === '/' && context.math === MATH.PARENS_DIVISION
+                ) {
                     return new Operation(this.op, [a, b], this.isSpaced);
                 }
                 throw { type: 'Operation',

diff --git a/packages/test-data/less/math/parens-division/media-math.less b/packages/test-data/less/math/parens-division/media-math.less
@@ -1,4 +1,4 @@
-@var: 16;
+@var: 10 + 6;
 
 @media (min-width: @var + 1) {
   .foo { bar: 1; }