Fixes stylus#834 Percentage calculation not used when value is expres…

…sion
tonistiigi · Oct 23, 2012 · dd3c3f7 · dd3c3f7
1 parent b4c438d
commit dd3c3f7
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 1 deletion.
diff --git a/lib/nodes/unit.js b/lib/nodes/unit.js
@@ -92,11 +92,12 @@ Unit.prototype.operate = function(op, right){
 
   // operate
   if (this.shouldCoerce(op)) {
+    right = right.first;
     // percentages
     if (('-' == op || '+' == op) && '%' == right.type) {
       right.val = this.val * (right.val / 100);
     } else {
-      right = this.coerce(right.first);
+      right = this.coerce(right);
     }
 
     switch (op) {

diff --git a/test/cases/regression.834.css b/test/cases/regression.834.css
@@ -0,0 +1,6 @@
+.grid {
+  width: 45%;
+}
+.grid {
+  width: 45%;
+}
diff --git a/test/cases/regression.834.styl b/test/cases/regression.834.styl
@@ -0,0 +1,8 @@
+grid(x)
+  width: 50% - x
+
+.grid
+  grid(10%)
+
+.grid
+  grid 10%