Merge pull request #46 from ignovak/leading-zero

option: leading-zero

Author: mishanga

.csscomb.json

@@ -4,6 +4,7 @@
         "node_modules/**"
     ],
     "colon-space": true,
+    "leading-zero": false,
     "rule-indent": true,
     "block-indent": true,
     "stick-brace": "\n",

README.md

@@ -35,6 +35,7 @@ Example configuration:
 {
     "exclude": ["node_modules/**"],
     "colon-space": true,
+    "leading-zero": false,
     "rule-indent": true,
     "block-indent": true,
     "stick-brace": true,

lib/csscomb.js

@@ -12,6 +12,7 @@ var Comb = function() {
     this._handlers;
     this._options = [
         'always-semicolon',
+        'leading-zero',
         'strip-spaces',
         'stick-brace',
         'colon-space',
@@ -35,7 +36,7 @@ Comb.prototype = {
         var _this = this;
         this._handlers = [];
         this._options.forEach(function(option) {
-            if (config[option]) {
+            if (config[option] !== undefined) {
                 try {
                     var handler = require('./options/' + option).setValue(config[option]);
                     handler && _this._handlers.push(handler);

lib/options/leading-zero.js

@@ -0,0 +1,32 @@
+module.exports = {
+
+    /**
+     * Sets handler value.
+     *
+     * @param {Boolean} value Option value
+     * @returns {Object}
+     */
+    setValue: function(value) {
+        if (value === true || value === false) {
+            this._value = value;
+            return this;
+        }
+    },
+
+    /**
+     * Processes tree node.
+     * @param {String} nodeType
+     * @param {node} node
+     */
+    process: function(nodeType, node) {
+        if (nodeType === 'number') {
+            if (this._value) {
+                if (node[0][0] === '.')
+                    node[0] = '0' + node[0];
+            } else {
+                node[0] = node[0].replace(/^0+(?=\.)/, '');
+            }
+        }
+    }
+
+};

test/integral.expect.css

@@ -4,11 +4,11 @@
     /* :before — бордер */
     .radio-button_theme_normal .radio-button__radio:before
     {
-        background: rgba(0,0,0,0.4);
-        background: -webkit-linear-gradient(top, rgba(0,0,0,0.2) 0,rgba(0,0,0,0.4) 100%);
-        background: -moz-linear-gradient(top, rgba(0,0,0,0.2) 0, rgba(0,0,0,0.4) 100%);
-        background: -o-linear-gradient(top, rgba(0,0,0,0.2) 0,rgba(0,0,0,0.4) 100%);
-        background: linear-gradient(to bottom, rgba(0,0,0,0.2) 0,rgba(0,0,0,0.4) 100%);
+        background: rgba(0,0,0,.4);
+        background: -webkit-linear-gradient(top, rgba(0,0,0,.2) 0,rgba(0,0,0,.4) 100%);
+        background: -moz-linear-gradient(top, rgba(0,0,0,.2) 0, rgba(0,0,0,.4) 100%);
+        background: -o-linear-gradient(top, rgba(0,0,0,.2) 0,rgba(0,0,0,.4) 100%);
+        background: linear-gradient(to bottom, rgba(0,0,0,.2) 0,rgba(0,0,0,.4) 100%);
         -moz-box-shadow: 0 1px 0 rgba(0,0,0,.07);
         box-shadow: 0 1px 0 rgba(0,0,0,.07);
     }
@@ -87,7 +87,7 @@ div p em
     {
         -moz-box-sizing: border-box;
         box-sizing: border-box;
-        padding: 0.4em 0;
+        padding: .4em 0;
 
         border: 0;
         outline: 0;

test/leading-zero.js

@@ -0,0 +1,31 @@
+var Comb = require('../lib/csscomb');
+var assert = require('assert');
+
+describe('options/leading-zero', function() {
+    var comb;
+
+    beforeEach(function() {
+        comb = new Comb();
+    });
+
+    it('Should add leading zero in dimensions', function() {
+        comb.configure({ 'leading-zero': true });
+        assert.equal(
+            comb.processString(
+                'div { margin: .5em }'
+            ),
+            'div { margin: 0.5em }'
+        );
+    });
+
+    it('Should remove leading zero in dimensions', function() {
+        comb.configure({ 'leading-zero': false });
+        assert.equal(
+            comb.processString(
+                'div { margin: 0.5em }'
+            ),
+            'div { margin: .5em }'
+        );
+    });
+
+});