less 2.4.0 → 2.5.3

package/.jscsrc

@@ -1,73 +1,73 @@
-{
-    "disallowImplicitTypeConversion": ["numeric", "boolean", "binary", "string"],
-    "disallowKeywords": ["with"],
-    "disallowMixedSpacesAndTabs": true,
-    "disallowMultipleLineBreaks": true,
-    "disallowOperatorBeforeLineBreak": ["."],
-    "disallowSpaceAfterKeywords": [
-                                        "void"//,
-                                        //"typeof"
-                                  ],
-    "disallowSpaceAfterPrefixUnaryOperators": ["++", "--", "+", "-", "~"],
-    "disallowSpaceBeforePostfixUnaryOperators": ["++", "--"],
-    "disallowSpacesInCallExpression": true,
-    "disallowSpacesInNamedFunctionExpression": {
-        "beforeOpeningRoundBrace": true},
-    "disallowTrailingComma": true,
-    "disallowTrailingWhitespace": true,
-    "maximumLineLength": 160,
-    "requireCommaBeforeLineBreak": true,
-    "requireCurlyBraces": [   "if",
-                              "else",
-                              "for",
-                              "while",
-                              "do",
-                              "try",
-                              "catch"],
-    "requireOperatorBeforeLineBreak": [ "?",
-                                        "=",
-                                        "+",
-                                        "-",
-                                        "/",
-                                        "*",
-                                        "==",
-                                        "===",
-                                        "!=",
-                                        "!==",
-                                        ">",
-                                        ">=",
-                                        "<",
-                                        "<="],
-    "requireSpaceAfterBinaryOperators": true,
-    "requireSpaceAfterKeywords": [
-                                        "else",
-                                        "case",
-                                        "try",
-                                        "typeof",
-                                        "return",
-                                        "if",
-                                        "for",
-                                        "while",
-                                        "do"
-                                    ],
-    "requireSpaceBeforeBlockStatements": true,
-    "requireSpaceBeforeBinaryOperators": [
-                                        "=",
-                                        "+",
-                                        "-",
-                                        "/",
-                                        "*",
-                                        "==",
-                                        "===",
-                                        "!=",
-                                        "!=="
-                                    ],
-    "requireSpaceBeforeBlockStatements": true,
-    "requireSpaceBetweenArguments": true,
-    "requireSpacesInConditionalExpression": true,
-    "requireSpacesInForStatement": true,
-    "requireSpacesInNamedFunctionExpression": {
-        "beforeOpeningCurlyBrace": true
-    },
-    "validateIndentation": 4
+{
+    "disallowImplicitTypeConversion": ["numeric", "boolean", "binary", "string"],
+    "disallowKeywords": ["with"],
+    "disallowMixedSpacesAndTabs": true,
+    "disallowMultipleLineBreaks": true,
+    "disallowOperatorBeforeLineBreak": ["."],
+    "disallowSpaceAfterKeywords": [
+                                        "void"//,
+                                        //"typeof"
+                                  ],
+    "disallowSpaceAfterPrefixUnaryOperators": ["++", "--", "+", "-", "~"],
+    "disallowSpaceBeforePostfixUnaryOperators": ["++", "--"],
+    "disallowSpacesInCallExpression": true,
+    "disallowSpacesInNamedFunctionExpression": {
+        "beforeOpeningRoundBrace": true},
+    "disallowTrailingComma": true,
+    "disallowTrailingWhitespace": true,
+    "maximumLineLength": 160,
+    "requireCommaBeforeLineBreak": true,
+    "requireCurlyBraces": [   "if",
+                              "else",
+                              "for",
+                              "while",
+                              "do",
+                              "try",
+                              "catch"],
+    "requireOperatorBeforeLineBreak": [ "?",
+                                        "=",
+                                        "+",
+                                        "-",
+                                        "/",
+                                        "*",
+                                        "==",
+                                        "===",
+                                        "!=",
+                                        "!==",
+                                        ">",
+                                        ">=",
+                                        "<",
+                                        "<="],
+    "requireSpaceAfterBinaryOperators": true,
+    "requireSpaceAfterKeywords": [
+                                        "else",
+                                        "case",
+                                        "try",
+                                        "typeof",
+                                        "return",
+                                        "if",
+                                        "for",
+                                        "while",
+                                        "do"
+                                    ],
+    "requireSpaceBeforeBlockStatements": true,
+    "requireSpaceBeforeBinaryOperators": [
+                                        "=",
+                                        "+",
+                                        "-",
+                                        "/",
+                                        "*",
+                                        "==",
+                                        "===",
+                                        "!=",
+                                        "!=="
+                                    ],
+    "requireSpaceBeforeBlockStatements": true,
+    "requireSpaceBetweenArguments": true,
+    "requireSpacesInConditionalExpression": true,
+    "requireSpacesInForStatement": true,
+    "requireSpacesInNamedFunctionExpression": {
+        "beforeOpeningCurlyBrace": true
+    },
+    "validateIndentation": 4
 }

package/.travis.yml

@@ -1,5 +1,6 @@
 language: node_js
 node_js:
+  - "4.0.0"
   - "0.12"
   - "0.11"
   - "0.10"

package/CHANGELOG.md

@@ -1,3 +1,49 @@
+# 2.5.3
+
+2015-09-25
+
+ - Fix import inline a URL
+
+# 2.5.2
+
+2015-09-24
+
+ - No output should result in an empty sourcemap
+ - Import inline located inside file imported by reference should not be present in output
+ - Shorthand colors will stay shorthand
+ - Make percentage work like other math functions and throw an error on NaN
+ - Fixed mixin definition matching problem when mixin definition contains parameters with default values
+ - Observe reference for imported comments
+
+# 2.5.1
+
+2015-05-21
+
+ - Fix problems with less being async in some browsers
+ - Minor fix only likely to affect programmatic usage of ruleset find
+ - Fix  error when a namespaced mixin is invoked in global scope
+
+# 2.5.0
+
+2015-04-03
+
+ - supports the scoped `@plugin` directive to load function plugins
+ - All directives are bubbled (e.g. supports), not just media
+ - Performance improvements to the parser - should help non-chrome browsers with very large less files to be a lot quicker.
+ - the image size function respects include paths like other file functions
+ - colour functions take a relative argument that applies percentages relatively instead of absolutely
+ - include paths now allows : as a separator on windows (recognising and not splitting drive names by the backslash)
+ - `@import (css)` does not pull the directive above comments
+ - Fix for import statements without quotes sometimes causing issues
+ - replace supports dimensions and colours
+ - the browser field is set in the package.json for use with browserify
+ - another fix to support paths being passed as a string instead of an array
+ - detached rulesets can be used as default arguments
+ - Fix a lot of false warnings about extends
+ - errors written to stderr more consistently
+ - consistently keep units if strict units is off
+ - Better support for comments in function all arguments
+
 # 2.4.0
 
 2015-02-07
@@ -5,7 +51,7 @@
  - Support for plugins that pre-process (to add libraries silently etc.)
  - Empty sourcemaps now work
  - Extract and Length functions now ignore comments in a list (more work to come to fix the general problem)
- - fragment url's are treated absolute since they refer to the html document
+ - fragment urls are treated absolute since they refer to the html document
  - Extends on a selector joined with `&` now work better
  - Nested mixins work better with !important (regression in 2.3.0)
  - The promise dependency is now actually optional (introduced in 2.0.0)
@@ -23,7 +69,7 @@
 
 2015-01-27
 
- - add isruleset function
+ - add `isruleset` function
  - add optional import option, causing less to not fail if file not found
  - Fix browsers-side cache.
  - Many fixes to import reference - support `@support` and keyframe
@@ -47,12 +93,12 @@
  - The node version of less now has image-size, image-width, image-height which return the image dimensions of a file
  - Fixed an issue that could cause the parse to occur more than once and the callback be called multiple times
  - if you are outputting to the console, lessc defaults to silent so warnings do not end up in output
- - isunit function supports '' to test if a dimension has no unit
+ - `isunit` function supports `''` to test if a dimension has no unit
  - data-uri function now counts characters after base64 encoding instead of bytes before encoding to determine ie8 support
  - fix bug effecting guards on pseudo class selectors
  - do not cache on the browser when used with modifyVars
  - detection if less does not parse last character in file
- - detection of whether a file is css now requires /css, .css, ?css, &css instead of just css. You can still tell less the type of file using import options.
+ - detection of whether a file is css now requires `/css`, `.css`, `?css`, `&css` instead of just `css`. You can still tell less the type of file using import options.
  - remove extra new line added to sourcemap entry inline file
  - support safari extension
  - less.parse now exposes a way to get the AST. We do not recommend you use this unless you need to.
@@ -69,7 +115,7 @@
 2014-11-27
 
  - Improved keyword and anonymous usage with the replace function
- - Added getCSSAppendage to sourcemap builder to avoid duplication in plugins
+ - Added `getCSSAppendage` to sourcemap builder to avoid duplication in plugins
  - Fix problem with plugins when used with the promises version of render
  - If the render callback throws an exception it now propogates instead of calling the callback again with an error
 
@@ -77,7 +123,7 @@
 
 2014-11-23
 
- - Fixed isSync option, it was using sync file operations but promises are guaranteed to call back async. We now support promises as a feature rather than the 1st class way of doing things.
+ - Fixed `isSync` option, it was using sync file operations but promises are guaranteed to call back async. We now support promises as a feature rather than the 1st class way of doing things.
  - Browser code is now synchronous again, like in v1, meaning it blocks the site until less is compiled
  - Some fixes for variable imports which affected filemanagers when synchronous
  - Fixed lessc makefile dependencies option
@@ -97,7 +143,7 @@
  - browser tests now pass in IE 8-11 and FF
  - added index.js and browser.js in root as shortcuts
  - fixed some local variable spellings
- - support for @counter-style directive
+ - support for `@counter-style` directive
 
 # 2.0.0-b3
 
@@ -106,7 +152,7 @@
  - some refactoring of browser structure to allow use of api vs normal browser bundle
  - browser bundle no longer leaks require
  - browser can now be scoped with just window
- - browser useFileCache defaults to true, but file cache is now cleared when refreshing or in watch mode
+ - browser `useFileCache` defaults to `true`, but file cache is now cleared when refreshing or in watch mode
 
 # 2.0.0-b2
 

package/Gruntfile.js

@@ -36,7 +36,7 @@ module.exports = function (grunt) {
                 command: 'node test'
             },
             benchmark: {
-                command: 'node benchmark/less-benchmark.js'
+                command: 'node benchmark/index.js'
             },
             "sourcemap-test": {
                 command: [
@@ -51,7 +51,6 @@ module.exports = function (grunt) {
                 src: ['./lib/less-browser/bootstrap.js'],
                 options: {
                     exclude: ["promise"],
-                    require: ["promise/polyfill.js"],
                     browserifyOptions: {
                         standalone: 'less'
                     }
@@ -251,13 +250,45 @@ module.exports = function (grunt) {
                     specs: 'test/browser/runner-postProcessor.js',
                     outfile: 'tmp/browser/test-runner-post-processor.html'
                 }
-            }
+            },
+            postProcessorPlugin: {
+                src: ['test/less/postProcessorPlugin/*.less'],
+                options: {
+                    helpers: ['test/plugins/postprocess/index.js','test/browser/runner-postProcessorPlugin-options.js'],
+                    specs: 'test/browser/runner-postProcessorPlugin.js',
+                    outfile: 'tmp/browser/test-runner-post-processor-plugin.html'
+                }
+            },
+            preProcessorPlugin: {
+                src: ['test/less/preProcessorPlugin/*.less'],
+                options: {
+                    helpers: ['test/plugins/preprocess/index.js','test/browser/runner-preProcessorPlugin-options.js'],
+                    specs: 'test/browser/runner-preProcessorPlugin.js',
+                    outfile: 'tmp/browser/test-runner-pre-processor-plugin.html'
+                }
+            },
+            visitorPlugin: {
+                src: ['test/less/visitorPlugin/*.less'],
+                options: {
+                    helpers: ['test/plugins/visitor/index.js','test/browser/runner-VisitorPlugin-options.js'],
+                    specs: 'test/browser/runner-VisitorPlugin.js',
+                    outfile: 'tmp/browser/test-runner-visitor-plugin.html'
+                }
+            },
+            filemanagerPlugin: {
+                src: ['test/less/filemanagerPlugin/*.less'],
+                options: {
+                    helpers: ['test/plugins/filemanager/index.js','test/browser/runner-filemanagerPlugin-options.js'],
+                    specs: 'test/browser/runner-filemanagerPlugin.js',
+                    outfile: 'tmp/browser/test-runner-filemanager-plugin.html'
+                }
+            }            
         },
 
         'saucelabs-jasmine': {
             all: {
                 options: {
-                    urls: ["post-processor", "global-vars", "modify-vars", "production", "rootpath-relative",
+                    urls: ["filemanager-plugin","visitor-plugin","pre-processor-plugin","post-processor-plugin","post-processor", "global-vars", "modify-vars", "production", "rootpath-relative",
                            "rootpath", "relative-urls", "browser", "no-js-errors", "legacy", "strict-units"
                     ].map(function(testName) {
                         return "http://localhost:8081/tmp/browser/test-runner-" + testName + ".html";
@@ -320,7 +351,7 @@ module.exports = function (grunt) {
     });
 
     // Load these plugins to provide the necessary tasks
-    require('matchdep').filterDev('grunt-*').forEach(grunt.loadNpmTasks);
+    require('jit-grunt')(grunt);
 
     // Actually load this plugin's task(s).
     grunt.loadTasks('build/tasks');
@@ -330,18 +361,11 @@ module.exports = function (grunt) {
         'test'
     ]);
 
-    grunt.registerTask('updateBowerJson', function () {
-        var bowerJson = require('./bower.json');
-        bowerJson.version = grunt.config('pkg.version');
-        fs.writeFileSync('./bower.json', JSON.stringify(bowerJson, null, 2));
-    });
-
     // Release
     grunt.registerTask('dist', [
         'browserify:browser',
         'concat:dist',
-        'uglify:dist',
-        'updateBowerJson'
+        'uglify:dist'
     ]);
 
     // Release Rhino Version
@@ -418,6 +442,9 @@ module.exports = function (grunt) {
     // Run all tests
     grunt.registerTask('test', testTasks);
 
+    // Run all tests
+    grunt.registerTask('quicktest', testTasks.slice(0, testTasks.length -1));
+
     // generate a good test environment for testing sourcemaps
     grunt.registerTask('sourcemap-test', [
         'clean:sourcemap-test',

package/README.md

@@ -4,6 +4,8 @@
 
 # [Less.js](http://lesscss.org)
 
+[![Join the chat at https://gitter.im/less/less.js](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/less/less.js?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
+
 > The **dynamic** stylesheet language. [http://lesscss.org](http://lesscss.org).
 
 This is the JavaScript, official, stable version of Less.

package/appveyor.yml

@@ -2,14 +2,8 @@
 environment:
   matrix:
     - nodejs_version: "0.10"
-    - nodejs_version: "0.11"
     - nodejs_version: "0.12"
 
-# Allow failing jobs for bleeding-edge Node.js versions.
-matrix:
-  allow_failures:
-    - nodejs_version: "0.11"
-
 # Install scripts. (runs after repo cloning)
 install:
   # Get the latest stable version of Node 0.STABLE.latest

package/bin/lessc

@@ -2,7 +2,7 @@
 
 var path = require('path'),
     fs = require('../lib/less-node/fs'),
-    os = require('os'),
+    os = require("os"),
     errno,
     mkdirp;
 
@@ -49,8 +49,9 @@ process.on('exit', function() { process.reallyExit(currentErrorcode); });
 
 var checkArgFunc = function(arg, option) {
     if (!option) {
-        console.log(arg + " option requires a parameter");
+        console.error(arg + " option requires a parameter");
         continueProcessing = false;
+        currentErrorcode = 1;
         return false;
     }
     return true;
@@ -59,8 +60,9 @@ var checkArgFunc = function(arg, option) {
 var checkBooleanArg = function(arg) {
     var onOff = /^((on|t|true|y|yes)|(off|f|false|n|no))$/i.exec(arg);
     if (!onOff) {
-        console.log(" unable to parse " + arg + " as a boolean. use one of on/t/true/y/yes/off/f/false/n/no");
+        console.error(" unable to parse " + arg + " as a boolean. use one of on/t/true/y/yes/off/f/false/n/no");
         continueProcessing = false;
+        currentErrorcode = 1;
         return false;
     }
     return Boolean(onOff[2]);
@@ -71,7 +73,6 @@ var parseVariableOption = function(option, variables) {
     variables[parts[0]] = parts[1];
 };
 
-var warningMessages = "";
 var sourceMapFileInline = false;
 
 function printUsage() {
@@ -152,7 +153,10 @@ function printUsage() {
                 break;
             case 'include-path':
                 if (checkArgFunc(arg, match[2])) {
-                    options.paths = match[2].split(os.type().match(/Windows/) ? ';' : ':')
+                    // ; supported on windows.
+                    // : supported on windows and linux, excluding a drive letter like C:\ so C:\file:D:\file parses to 2
+                    options.paths = match[2]
+                        .split(os.type().match(/Windows/) ? /:(?!\\)|;/ : ':')
                         .map(function(p) {
                             if (p) {
                                 return path.resolve(process.cwd(), p);
@@ -246,10 +250,8 @@ function printUsage() {
                 if (plugin) {
                     plugins.push(plugin);
                 } else {
-                    console.log("Unable to load plugin " + name +
+                    console.error("Unable to load plugin " + name +
                         " please make sure that it is installed under or at the same level as less");
-                    console.log();
-                    printUsage();
                     currentErrorcode = 1;
                 }
                 break;
@@ -258,11 +260,9 @@ function printUsage() {
                 if (plugin) {
                     plugins.push(plugin);
                 } else {
-                    console.log("Unable to interpret argument " + arg +
+                    console.error("Unable to interpret argument " + arg +
                         " - if it is a plugin (less-plugin-" + arg + "), make sure that it is installed under or at" +
                         " the same level as less");
-                    console.log();
-                    printUsage();
                     currentErrorcode = 1;
                 }
                 break;
@@ -281,9 +281,6 @@ function printUsage() {
     var outputbase = args[2];
     if (output) {
         output = path.resolve(process.cwd(), output);
-        if (warningMessages) {
-            console.log(warningMessages);
-        }
     }
 
     if (options.sourceMap) {
@@ -325,8 +322,8 @@ function printUsage() {
     }
 
     if (! input) {
-        console.log("lessc: no input files");
-        console.log("");
+        console.error("lessc: no input files");
+        console.error("");
         printUsage();
         currentErrorcode = 1;
         return;
@@ -356,6 +353,7 @@ function printUsage() {
 
     if (!sourceMapFileInline) {
         var writeSourceMap = function(output, onDone) {
+            output = output || "";
             var filename = sourceMapOptions.sourceMapFullFilename;
             ensureDirectory(filename);
             fs.writeFile(filename, output, 'utf8', function (err) {
@@ -366,8 +364,8 @@ function printUsage() {
                     } else {
                         description += err.code + " " + err.message;
                     }
-                    less.logger.error('lessc: failed to create file ' + filename);
-                    less.logger.error(description);
+                    console.error('lessc: failed to create file ' + filename);
+                    console.error(description);
                 } else {
                     less.logger.info('lessc: wrote ' + filename);
                 }
@@ -395,8 +393,8 @@ function printUsage() {
                     } else {
                         description += err.code + " " + err.message;
                     }
-                    less.logger.error('lessc: failed to create file ' + output);
-                    less.logger.error(description);
+                    console.error('lessc: failed to create file ' + output);
+                    console.error(description);
                 } else {
                     less.logger.info('lessc: wrote ' + output);
                     onSuccess();
@@ -420,7 +418,7 @@ function printUsage() {
 
     var parseLessFile = function (e, data) {
         if (e) {
-            console.log("lessc: " + e.message);
+            console.error("lessc: " + e.message);
             currentErrorcode = 1;
             return;
         }
@@ -446,13 +444,13 @@ function printUsage() {
                 }
             },
             warn: function(msg) {
-                // do not show warning if outputting css to the console or the silent option is used
-                if (!silent && output) {
+                // do not show warning if the silent option is used
+                if (!silent) {
                     console.warn(msg);
                 }
             },
             error: function(msg) {
-                console.log(msg);
+                console.error(msg);
             }
         });
 

package/bower.json

@@ -1,6 +1,5 @@
 {
   "name": "less",
-  "version": "2.4.0",
   "main": "dist/less.js",
   "ignore": [
     "**/.*",
@@ -22,4 +21,4 @@
     ".jshintrc",
     ".npmignore"
   ]
-}
+}