less 2.5.1 → 2.6.1

package/test/less-bom/errors/parse-error-media-no-block-3.txt

@@ -1,3 +1,3 @@
-ParseError: media definitions require block statements after any features in {path}parse-error-media-no-block-3.less on line 4, column 4:
+SyntaxError: media definitions require block statements after any features in {path}parse-error-media-no-block-3.less on line 4, column 4:
 3     font-size: 5000px;
 4   }

package/test/less-bom/errors/percentage-non-number-argument.less

@@ -0,0 +1,3 @@
+div {
+  percentage: percentage(16/17);
+}

package/test/less-bom/errors/percentage-non-number-argument.txt

@@ -0,0 +1,4 @@
+ArgumentError: error evaluating function `percentage`: argument must be a number in {path}percentage-non-number-argument.less on line 2, column 15:
+1 div {
+2   percentage: percentage(16/17);
+3 }

package/test/less-bom/errors/property-interp-not-defined.txt

@@ -1,2 +1,2 @@
-NameError: variable @color is undefined in {path}property-interp-not-defined.less on line 1, column 12:
-1 a {outline-@{color}: green}
+NameError: variable @color is undefined in {path}property-interp-not-defined.less on line 1, column 12:
+1 a {outline-@{color}: green}

package/test/less-bom/extend-selector.less

@@ -96,4 +96,15 @@ div.ext5,
   .footer-nav {
     &:extend( .header .header-nav all );
   }
-}
+}
+
+.issue-2586-bordered {
+    border: solid 1px black;
+}
+.issue-2586-somepage {
+    .content:extend(.issue-2586-bordered) {
+        &>span {
+            margin-bottom: 10px;
+        }
+    }
+}

package/test/less-bom/import/import-inline-invalid-css.less

@@ -0,0 +1 @@
+@import (inline) "invalid-css.less";

package/test/less-bom/import/import-reference.less

@@ -37,7 +37,7 @@
 
 .zz {
   .y {
-    pulled-in: yes;
+    pulled-in: yes /* inline comment survives */;
   }
   /* comment pulled in */
 }
@@ -86,4 +86,13 @@
   }
   @rules1: {property: value;};
   @rules2: {property: value;};
+}
+
+@import (inline, multiple) "invalid-css.less";
+@import "import-inline-invalid-css.less";
+.print-referenced-import-inline() {
+  div {
+    @import (inline, multiple) "invalid-css.less";
+  }
+  @import (inline, multiple) "invalid-css.less";
 }

package/test/less-bom/import-reference-issues/appender-reference-1968.less

@@ -0,0 +1,6 @@
+.container-no-parent-selector {
+   .mixin-no-parent-selector();
+}
+.container-with-parent-selector {
+  .mixin-with-parent-selector()
+}

package/test/less-bom/import-reference-issues/global-scope-import.less

@@ -0,0 +1,13 @@
+.theOnlySelector {
+  avoid: warning;
+}
+.unusedAndReference:extend(.theOnlySelector) {
+    unused-and: reference;
+}
+
+@import (reference) "global-scope-nested.less";
+
+.test-b {
+  background-color: green;
+  &:extend(.test-a all);
+}

package/test/less-bom/import-reference-issues/global-scope-nested.less

@@ -0,0 +1,3 @@
+.test-a {
+  color: red;
+}

package/test/less-bom/import-reference-issues/mixin-1968.less

@@ -0,0 +1,8 @@
+.mixin-no-parent-selector() {
+  background-color: red;
+}
+.mixin-with-parent-selector() {
+  &:first-child{ //  USING AN & SELECTOR HERE
+    background-color: blue;
+ }
+}

package/test/less-bom/import-reference-issues/multiple-import-nested.less

@@ -0,0 +1,12 @@
+/* double nested file */
+should {
+  be: invisible;
+}
+/* do not extend outside of this */
+.something {
+  invisible: suppress warning;
+}
+.invisible {
+ &:extend(.something all);
+}
+

package/test/less-bom/import-reference-issues/multiple-import.less

@@ -0,0 +1,10 @@
+/*
+  tralala
+*/
+.fix {
+  fix: fix;
+}
+.something {
+  @import (reference) "multiple-import-nested.less";
+  inside: something;
+}

package/test/less-bom/import-reference-issues/simple-mixin.css

@@ -0,0 +1,3 @@
+.mixin {
+  was: included;
+}

package/test/less-bom/import-reference-issues/simple-ruleset-2162.less

@@ -0,0 +1,3 @@
+ruleset {
+  shall-be-invisible: less;
+}

package/test/less-bom/import-reference-issues.less

@@ -0,0 +1,50 @@
+// Tests following past issues:
+// * #1851 - Extend within (reference) imported files 
+// * #1896 - Namespace imported less code does not get properly referenced, when there's a (reference) keyword.
+// * #1878 - extend inside referenced file should not extend outside selectors
+// * #2716 - A file imported by reference and then normally - multiple imports should be independent
+// * #1968 - When using an @import (reference), mixins that contain an & selector get added to the compiled output improperly
+// * #2162 - Cannot put import by reference inside of a mixin (also doubles #1896)
+
+// #1878: extend inside referenced file should not extend outside selectors
+@import (reference) "import-reference-issues/global-scope-import.less";
+.theOnlySelector {
+  shall-have: one selector;
+}
+
+// #2716: A file imported by reference and then normally - multiple imports should be independent
+// #1878: - double nested version
+#do-not-show-import {
+  @import (reference, multiple) "import-reference-issues/multiple-import.less";
+}
+
+show-all-content {
+  @import (multiple) "import-reference-issues/multiple-import.less";
+}
+// #1896: Namespace imported less code does not get properly referenced, when there's a (reference) keyword.
+#Namespace {
+  @import (less, reference) "import-reference-issues/simple-mixin.css";
+}
+#used-namespaced-mixin {
+  #Namespace > .mixin();
+  shall-see: another property above;
+}
+
+// #1851: Extend within (reference) imported files 
+// test-b is in global-scope-import.less file
+.test-c {
+  &:extend(.test-b all);
+}
+
+// #1968: When using an @import (reference), mixins that contain an & selector get added to the compiled output improperly
+@import "import-reference-issues/mixin-1968.less";
+@import (reference) "import-reference-issues/appender-reference-1968.less";
+
+// #2162 - Cannot put import by reference inside of a mixin (also doubles #1896)
+.mixin-with-import-by-reference-inside() {
+  the-only-property: nothing-below-this;
+  @import (reference) "import-reference-issues/simple-ruleset-2162.less";
+}
+call-mixin-with-import-by-reference-inside {
+  .mixin-with-import-by-reference-inside();
+}

package/test/less-bom/import-reference.less

@@ -22,3 +22,5 @@
 }
 .mixin-with-nested-selectors();
 .mixin-with-directives(some-name);
+
+.print-referenced-import-inline();

package/test/less-bom/mixins-args.less

@@ -212,4 +212,52 @@ body {
 
 div {
   .test-calling-one-arg-mixin(1);
+}
+
+mixins-args-expand-op- {
+  @x: 1, 2, 3;
+  @y: 4  5  6;
+
+  &1 {.m3(@x...)}
+  &2 {.m3(@y...)}
+  &3 {.wr(a, b, c)}
+  &4 {.wr(a; b; c, d)}
+  &5 {.wr(@x...)}
+  &6 {.m4(0; @x...)}
+  &7 {.m4(@x..., @a: 0)}
+  &8 {.m4(@b: 1.5; @x...)}
+  &9 {.aa(@y, @x..., and again, @y...)}
+
+  .m3(@a, @b, @c) {
+    m3: @a, @b, @c;
+  }
+
+  .m4(@a, @b, @c, @d) {
+    m4: @a, @b, @c, @d;
+  }
+
+  .wr(@a...) {
+    &a {.m3(@a...)}
+    &b {.m4(0, @a...)}
+    &c {.m4(@a..., 4)}
+  }
+
+  .aa(@a...) {
+    aa: @a;
+    a4: extract(@a, 5);
+    a8: extract(@a, 8);
+  }
+}
+#test-mixin-matching-when-default-2645 {
+  .mixin(@height) {
+    height: @height;
+  }
+
+  .mixin(@width, @height: 10px) {
+    width: @width;
+
+    .mixin(@height: @height);
+  }
+
+  .mixin(@height: 20px);
 }

package/test/less-bom/mixins-guards.less

@@ -278,3 +278,81 @@
     .guarded-mixin-for-root() when (@guarded-mixin-for-root) {}
 }
 #ns > .guarded-mixin-for-root();
+// various combinations of nested or, and, paranthesis and negation
+.parenthesisNot(@value) when ((((@value)))) {
+  parenthesisNot: just-value;
+}
+.parenthesisNot(@value) when (((not(@value)))) {
+  parenthesisNot: negated once inside;
+}
+.parenthesisNot(@value) when not((((@value)))) {
+  parenthesisNot: negated once outside;
+}
+.parenthesisNot(@value) when ((not((@value)))) {
+  parenthesisNot: negated once middle;
+}
+.parenthesisNot(@value) when not(((not(@value)))) {
+  parenthesisNot: negated twice 1;
+}
+.parenthesisNot(@value) when (not((not(@value)))) {
+  parenthesisNot: negated twice 2;
+}
+.parenthesisNot(@value) when ((not(not(@value)))) {
+  parenthesisNot: negated twice 3;
+}
+.parenthesisNot (...) when (default()) {
+  parenthesisNot: none matched;
+}
+
+#parenthesisNot-true {
+  .parenthesisNot(true);
+}
+#parenthesisNot-false {
+  .parenthesisNot(false);
+}
+
+.orderOfEvaluation(@a1, @a2, @a3) when ((@a1) and (@a2) or (@a3)) {
+  no-parenthesis: evaluated true 1a;
+}
+.orderOfEvaluation(@a1, @a2, @a3) when ((@a3) or (@a1) and (@a2)) {
+  no-parenthesis: evaluated true 1b;
+}
+.orderOfEvaluation(@a1, @a2, @a3) when ((@a1) and ((@a2) or (@a3))) {
+  no-parenthesis: evaluated true 1c;
+}
+.orderOfEvaluation(@a1, @a2, @a3) when (@a3), (@a1) and (@a2) {
+  no-parenthesis: evaluated true 1d;
+}
+.orderOfEvaluation(@a1, @a2, @a3) when (((@a3) or (@a1)) and (@a2)) {
+  no-parenthesis: evaluated true 1e;
+}
+.orderOfEvaluation(@a1, @a2, @a3) when ((@a1) and (@a2) or not (@a3)) {
+  no-parenthesis: evaluated true 2a;
+}
+.orderOfEvaluation(@a1, @a2, @a3) when (not (@a3) or (@a1) and (@a2)) {
+  no-parenthesis: evaluated true 2b;
+}
+.orderOfEvaluation(@a1, @a2, @a3) when not (@a3), (@a1) and (@a2) {
+  no-parenthesis: evaluated true 2c;
+}
+.orderOfEvaluation(@a1, @a2, @a3) when (not (@a1) and (@a2) or (@a3)) {
+  no-parenthesis: evaluated true 3;
+}
+.orderOfEvaluation(@a1, @a2, @a3) when ((((@a1) and (@a2) or (@a3)))) {
+  no-parenthesis: evaluated true 4;
+}
+.orderOfEvaluation(@a1, @a2, @a3) when (((@a1) and (@a2)) or (@a3)) {
+  with-parenthesis: evaluated true;
+}
+.orderOfEvaluation(...) when (default()) {
+  orderOfEvaluation: evaluated false;
+}
+#orderOfEvaluation-false-false-true {
+  .orderOfEvaluation(false, false, true);
+}
+#orderOfEvaluation-false-false-false {
+  .orderOfEvaluation(false, false, false);
+}
+#orderOfEvaluation-true-true-false {
+  .orderOfEvaluation(true, true, false);
+}

package/test/less-bom/no-strict-math/mixins-guards.less

@@ -0,0 +1,25 @@
+// https://github.com/less/less.js/issues/2798
+.test-2798 when ((8+4) < 13) { 
+  regression: fixed; 
+}
+.test-2798 when ((8+6) < 13) { 
+  regression: should not be visible; 
+}
+.conditions-parser-1 when (8+4 < 13) {
+  only-atomic: ok;
+}
+.conditions-parser-1 when (8+6 < 13) {
+  only-atomic: should not be visible;
+}
+.conditions-parser-2 when (8+(5-1) < 13) {
+  only-atomic-with-nested-parenthesis: ok;
+}
+.conditions-parser-2 when (8+(15-1) < 13) {
+  only-atomic-with-nested-parenthesis: should not be visible;
+}
+.conditions-parser-3 when (8 < (13+1)) {
+  only-atomic-nested-parenthesis-on-right: ok;
+}
+.conditions-parser-3 when (8 < (3+1)) {
+  only-atomic-nested-parenthesis-on-right: should not be visible;
+}

package/test/less-bom/no-strict-math/no-sm-operations.less

@@ -0,0 +1,10 @@
+.named-colors-in-expressions {
+    color-0: 0 -red;  
+    color-1: 1 - red; 
+    color-2: red * 2;
+    color-3: 2 * red;
+    @3: -red;
+    &-bar@{3} {x: y}
+    @color: red;
+    &-bar@{color} {a: a}
+}

package/test/less-bom/operations.less

@@ -1,5 +1,6 @@
 #operations {
   color: (#110000 + #000011 + #001100); // #111111
+  color-2: (yellow - #070707);
   height: (10px / 2px + 6px - 1px * 2); // 9px
   width: (2 * 4 - 5em); // 3em
   .spacing {

package/test/less-test.js

@@ -239,7 +239,12 @@ module.exports = function() {
                     fail("ERROR: " + (err && err.message));
                     if (isVerbose) {
                         process.stdout.write("\n");
-                        process.stdout.write(err.stack + "\n");
+                        if (err.stack) {
+                            process.stdout.write(err.stack + "\n");
+                        } else {
+                            //this sometimes happen - show the whole error object
+                            console.log(err);
+                        }
                     }
                     release();
                     return;

package/.idea/.name

@@ -1 +0,0 @@
-less.js

package/.idea/jsLibraryMappings.xml

@@ -1,6 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project version="4">
-  <component name="JavaScriptLibraryMappings">
-    <file url="file://$PROJECT_DIR$" libraries="{less.js node_modules}" />
-  </component>
-</project>

package/.idea/less.js.iml

@@ -1,9 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<module type="WEB_MODULE" version="4">
-  <component name="NewModuleRootManager">
-    <content url="file://$MODULE_DIR$" />
-    <orderEntry type="inheritedJdk" />
-    <orderEntry type="sourceFolder" forTests="false" />
-    <orderEntry type="library" name="less.js node_modules" level="project" />
-  </component>
-</module>

package/.idea/libraries/less_js_node_modules.xml

@@ -1,14 +0,0 @@
-<component name="libraryTable">
-  <library name="less.js node_modules" type="javaScript">
-    <properties>
-      <option name="frameworkName" value="node_modules" />
-      <sourceFilesUrls>
-        <item url="file://$PROJECT_DIR$/node_modules" />
-      </sourceFilesUrls>
-    </properties>
-    <CLASSES>
-      <root url="file://$PROJECT_DIR$/node_modules" />
-    </CLASSES>
-    <SOURCES />
-  </library>
-</component>

package/.idea/misc.xml

@@ -1,14 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project version="4">
-  <component name="ProjectLevelVcsManager" settingsEditedManually="false">
-    <OptionsSetting value="true" id="Add" />
-    <OptionsSetting value="true" id="Remove" />
-    <OptionsSetting value="true" id="Checkout" />
-    <OptionsSetting value="true" id="Update" />
-    <OptionsSetting value="true" id="Status" />
-    <OptionsSetting value="true" id="Edit" />
-    <ConfirmationsSetting value="0" id="Add" />
-    <ConfirmationsSetting value="0" id="Remove" />
-  </component>
-  <component name="ProjectRootManager" version="2" />
-</project>

package/.idea/modules.xml

@@ -1,8 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project version="4">
-  <component name="ProjectModuleManager">
-    <modules>
-      <module fileurl="file://$PROJECT_DIR$/.idea/less.js.iml" filepath="$PROJECT_DIR$/.idea/less.js.iml" />
-    </modules>
-  </component>
-</project>

package/.idea/vcs.xml

@@ -1,6 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project version="4">
-  <component name="VcsDirectoryMappings">
-    <mapping directory="$PROJECT_DIR$" vcs="Git" />
-  </component>
-</project>