less 3.10.1 → 3.11.1

package/test/less/mixins-guards.less

@@ -2,7 +2,7 @@
 // Stacking, functions..
 
 .light (@a) when (lightness(@a) > 50%) {
-  color: white;
+  color: inherit;
 }
 .light (@a) when (lightness(@a) < 50%) {
   color: black;
@@ -63,18 +63,18 @@
 .default1 { .default }
 
 // true & false keywords
-.test (@a) when (@a) {
+.test-rule (@a) when (@a) {
     content: "true.";
 }
-.test (@a) when not (@a) {
+.test-rule (@a) when not (@a) {
     content: "false.";
 }
 
-.test1 { .test(true) }
-.test2 { .test(false) }
-.test3 { .test(1) }
-.test4 { .test(boo) }
-.test5 { .test("true") }
+.test-rule1 { .test-rule(true) }
+.test-rule2 { .test-rule(false) }
+.test-rule3 { .test-rule(1) }
+.test-rule4 { .test-rule(boo) }
+.test-rule5 { .test-rule("true") }
 
 // Boolean expressions
 

package/test/less/mixins-important.less

@@ -35,7 +35,7 @@
 .when-calling-nested-with-param-issue-2394 {
   .size(10px) !important;
 }
-.testMixin-2421 () {
+.test-ruleMixin-2421 () {
     .topCheck-2421 () {
         .nestedCheck-2421() {
             margin: 5px;
@@ -45,9 +45,9 @@
     .topCheck-2421();
 }
 .class1-2421 {
-    .testMixin-2421() !important;
+    .test-ruleMixin-2421() !important;
 }
 .class2-2421 {
-    .testMixin-2421();
+    .test-ruleMixin-2421();
 }
 

package/test/less/mixins.less

@@ -28,7 +28,7 @@
 
 #header {
   .milk {
-    color: white;
+    color: inherit;
     .mixin;
     #theme > .mixin;
   }
@@ -119,7 +119,7 @@ h3 { .margin_between(15px, 5px); }
 .recursion() {
   color: black;
 }
-.test-rec {
+.test-rule-rec {
   .recursion {
     .recursion();
   }

package/test/less/namespacing/namespacing-3.less

@@ -2,7 +2,7 @@
   @width: 400px;
   @colors: {
     toolbar-background: red;
-    toolbar-foreground: white;
+    toolbar-foreground: inherit;
   }
 };
 
@@ -25,4 +25,27 @@
     background: @map[@colors][toolbar-background];
     color: @map[@colors][toolbar-foreground];
   }
-}
+}
+
+// !important after map usage
+// https://github.com/less/less.js/issues/3430
+@margins: {
+    zero: 0;
+    ten: 10px;
+}
+.cell {
+  margin: @margins[zero] @margins[ten]/2 !important;
+}
+
+.mixin(@color: black; @margin: 10px; @padding: 20px) {
+  color: @color;
+  margin: @margin;
+  padding: @padding;
+  width: @margins[zero] !important
+}
+.class1 {
+  .mixin(@margin: 20px; @color: #33acfe) !important;
+}
+.class2 {
+  .mixin(#efca44; @padding: 40px 10px);
+}

package/test/less/namespacing/namespacing-4.less

@@ -12,7 +12,7 @@
   .core() {
     .colors() {
       primary: blue;
-      foreground: white;
+      foreground: inherit;
     }
   }
 }

package/test/less/namespacing/namespacing-5.less

@@ -8,7 +8,7 @@
     primary: black;
     secondary: grey;
   }
-  .test() {
+  .test-rule() {
     val: output;
   }
 }
@@ -16,7 +16,7 @@
 .my-navbar {
   #theme.dark.navbar();
   background: .colors[primary];
-  .test();
+  .test-rule();
 }
 
 .another-navbar {

package/test/less/namespacing/namespacing-functions.less

@@ -8,4 +8,30 @@
 
 @return: {
   single: val;
+}
+
+// Issue #3405
+#lookup {
+  @prop: test;
+}
+
+.mix (@var) {
+  width: @var;
+}
+
+.bar {
+  .mix(#lookup[@prop]);
+}
+
+// Issue #3406
+.mix2 (@n) {
+  value: @n;
+}
+#lookup2 {
+  @var: .mix2(lookup);
+}
+.example {
+  // #lookup[@var](); -- fails, need the following alias
+  @dr: #lookup2[@var];
+  @dr();
 }

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

@@ -1,8 +1,8 @@
 // https://github.com/less/less.js/issues/2798
-.test-2798 when ((8+4) < 13) { 
+.test-rule-2798 when ((8+4) < 13) { 
   regression: fixed; 
 }
-.test-2798 when ((8+6) < 13) { 
+.test-rule-2798 when ((8+6) < 13) { 
   regression: should not be visible; 
 }
 .conditions-parser-1 when (8+4 < 13) {

package/test/less/permissive-parse.less

@@ -44,7 +44,7 @@
   }
 }
 // @todo - fix comment absorption after property
-.test-comment {
+.test-rule-comment {
   --value: a/* { ; } */;
   --comment-within: ( /* okay?; comment; */ );
 }

package/test/less/plugin.less

@@ -57,7 +57,7 @@
 
 // `test-global` function should propagate into directive scope
 @media screen {
-  .test {
+  .test-rule {
     result : test-global();
   }
 }
@@ -68,13 +68,13 @@
 // `test-global` function should propagate into nested directive scopes
 @media screen and (min-width:100px) {
   @media (max-width:400px) {
-		.test {
+		.test-rule {
 			result : test-global();
 		}
 	}
 }
 
-.test {
+.test-rule {
   @media screen {
     @plugin "./plugin/plugin-local";
     result : test-local();
@@ -86,29 +86,29 @@
 .root {
   @ruleset2();
 }
-.test-empty {
+.test-rule-empty {
   val1: foo;
   test-collapse();
   val2: foo;
 }
-.test-simple {
+.test-rule-simple {
   @plugin "./plugin/plugin-simple";
   value: pi-anon();
   value: (pi() * 2);
 }
-.test-conflicts {
+.test-rule-conflicts {
   @plugin "./plugin/plugin-scope1";
   value: foo();
 }
-.test-conflicts {
+.test-rule-conflicts {
   @plugin "./plugin/plugin-scope2";
   value: foo();
 }
-.test-conflicts {
+.test-rule-conflicts {
   @plugin "./plugin/plugin-scope1";
   value: foo();
 }
-.test-collection {
+.test-rule-collection {
   @plugin "./plugin/plugin-collection";
   @var: 32;
   store(@var);

package/test/less/postProcessorPlugin/postProcessor.less

@@ -1,4 +1,4 @@
-@color: white;
-.test {
+@color: inherit;
+.test-rule {
     color: @color;
 }

package/test/less/preProcessorPlugin/preProcessor.less

@@ -1,3 +1,3 @@
-.test {
+.test-rule {
     color: @color;
 }

package/test/less/selectors.less

@@ -125,7 +125,7 @@ a {
 :nth-child(@{num}) {
   selector: interpolated;
 }
-.test {
+.test-rule {
   &:nth-child(@{num}) {
     selector: interpolated;
   }

package/test/less/sourcemaps/imported.css

@@ -1,6 +1,6 @@
 /*comments*/
 .unused-css {
-    color: white;
+    color: inherit;
 }
 .imported {
     color: black;

package/test/less/variables.less

@@ -86,7 +86,7 @@
 .polluteMixin() {
     @a: 'pollution';
 }
-.testPollution {
+.test-rulePollution {
     @a: 'no-pollution';
     a: @a;
     .polluteMixin();

package/test/less/visitorPlugin/visitor.less

@@ -1,4 +1,4 @@
-.test {
+.test-rule {
     color: red;
     -some-aribitrary-property: value;
 }

package/test/less-test.js

@@ -7,15 +7,7 @@ module.exports = function() {
         doBomTest = false,
         clone = require('clone');
 
-    var less;
-
-    // Dist fallback for NPM-installed Less (for plugins that do testing)
-    try {
-        less = require('../tmp/less.cjs.js');
-    }
-    catch (e) {
-        less = require('../dist/less.cjs.js');
-    }
+    var less = require('./less');
 
     var stylize = require('../lib/less-node/lessc-helper').stylize;
 

package/test/less.js

@@ -0,0 +1,11 @@
+var less;
+
+// Dist fallback for NPM-installed Less (for plugins that do testing)
+try {
+    less = require('../tmp/less.cjs.js');
+}
+catch (e) {
+    less = require('../dist/less.cjs.js');
+}
+
+module.exports = less;

package/test/browser/onload.js

@@ -1,13 +0,0 @@
-(function() {
-    window.DEFER = [window.onload];
-    Object.defineProperty(window, 'onload', {
-        get: function() {
-            return function() {};
-        },
-        set: function(fn) {
-            window.DEFER.push(fn);
-        },
-        enumerable: true,
-        configurable: true
-    });
-})();

package/test/browser/test-runner-template.tmpl

@@ -1,105 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-  <head>
-    <meta charset="utf-8">
-    
-    <title>Jasmine Spec Runner</title>
-
-    <!-- for each test, generate CSS/LESS link tags -->
-    <% scripts.src.forEach(function(fullLessName) {
-      var pathParts = fullLessName.split('/');
-      var fullCssName = fullLessName.replace(/less/g, 'css');
-      var lessName = pathParts[pathParts.length - 1];
-      var name = lessName.split('.')[0]; %>
-      <!-- the tags to be generated -->
-      <link id="original-less:test-less-<%= name %>" title="test-less-<%= name %>" rel="stylesheet/less" type="text/css" href="<%= fullLessName %>">
-      <link id="expected-less:test-less-<%= name %>" rel="stylesheet" type="text/css" href="<%= fullCssName %>">
-    <% }); %>
-
-    <!-- generate grunt-contrib-jasmine link tags -->
-    <% css.forEach(function(style){ %>
-      <link rel="stylesheet" type="text/css" href="<%= style %>">
-    <% }) %>
-
-    <!-- generate script tags for tests -->
-    <% 
-    
-    var generateScriptTags = function(allScripts) {
-        allScripts.forEach(function(script){
-            if (script.indexOf('boot.js') > -1) {
-                script = '../../test/browser/vendor/boot.js'
-            }
-            %><script src="<%= script %>"></script> <%
-        });
-    }; 
-    
-    generateScriptTags(scripts.polyfills);
-    generateScriptTags(scripts.jasmine);
-    generateScriptTags(scripts.boot);
-    generateScriptTags(scripts.helpers);
-    generateScriptTags(scripts.vendor);
-
-    %>
-
-    <!-- generate script tags for tests -->
-    <% var toArray = function(scripts) {
-        %>[<%
-        scripts.forEach(function(scriptUrl, index){
-            %>"<%= scriptUrl %>"<%
-            if (index !== scripts.length -1) {
-                %>,<%
-            }
-        });
-        %>]<%
-        }; %>
-
-    <script>
-
-        function loadScript(url,callback){
-            var script = document.createElement('script');
-
-            if(document.documentMode === 8){
-                script.onreadystatechange = function(){
-                    if (script.readyState === 'loaded'){
-                        if (callback){callback()};
-                    };
-                };
-            } else {
-                script.onload = function(){
-                    if (callback){callback()};
-                };
-            };
-            script.src = url;
-            document.body.appendChild(script);
-        };
-
-            var specs = <% toArray(scripts.specs) %>,
-                reporters = <% toArray([].concat(scripts.reporters)) %>,
-                runScripts = specs.concat(reporters);
-        </script>
-
-       <script>
-        window.addEventListener('load', function() {
-
-            function addNextScript() {
-                // for sauce, see above. Additional step needed between loading jasmine and loading
-                // the js reporter
-                if (runScripts.length) {
-                    var scriptSrc = runScripts.shift();
-                    loadScript(scriptSrc, addNextScript);
-                }
-                else {
-                    // deferred boot function
-                    window.DEFER();
-                }
-            }
-            addNextScript();
-
-        });
-    </script>
-  </head>
-
-  <body>
-  <!-- content -->
-  </body>
-</html>