Added element-case option, closes #58

Author: kizu

.csscomb.json

@@ -8,6 +8,7 @@
     "colon-space": true,
     "color-case": "lower",
     "color-shorthand": true,
+    "element-case": "lower",
     "leading-zero": false,
     "rule-indent": true,
     "stick-brace": "\n",

README.md

@@ -44,6 +44,7 @@ Example configuration:
     "colon-space": true,
     "color-case": "lower",
     "color-shorthand": true,
+    "element-case": "lower",
     "leading-zero": false,
     "rule-indent": true,
     "stick-brace": true,
@@ -163,6 +164,20 @@ b { color: #ffcc00 }
 b { color: #fc0 }
 ```
 
+### element-case
+
+Available values: `{String}` `lower` or `upper`
+
+Example: `{ "element-case": "upper" }`
+
+```css
+/* before */
+li > a { color: red }
+
+/* after */
+LI > A { color: red }
+```
+
 ### leading-zero
 
 Available values: `{Boolean}` `true` or `false`

lib/csscomb.js

@@ -14,6 +14,7 @@ var Comb = function() {
         'always-semicolon',
         'color-case',
         'color-shorthand',
+        'element-case',
         'leading-zero',
         'strip-spaces',
         'stick-brace',

lib/options/element-case.js

@@ -0,0 +1,36 @@
+module.exports = {
+
+    /**
+     * Sets handler value.
+     *
+     * @param {String} value Option value
+     * @returns {Object|undefined}
+     */
+    setValue: function(value) {
+        if (value === 'lower' || value === 'upper') {
+            this._value = value;
+            return this;
+        }
+    },
+
+    /**
+     * Processes tree node.
+     * @param {String} nodeType
+     * @param {node} node
+     */
+    process: function(nodeType, node) {
+        if (nodeType === 'simpleselector') {
+            for (var i = node.length; i--;) {
+                var nodeItem = node[i];
+                if (nodeItem[0] === 'ident') {
+                    if (this._value === 'lower') {
+                        nodeItem[1] = nodeItem[1].toLowerCase();
+                    } else if (this._value === 'upper') {
+                        nodeItem[1] = nodeItem[1].toUpperCase();
+                    }
+                }
+            }
+        }
+    }
+
+};

test/element-case.js

@@ -0,0 +1,51 @@
+var Comb = require('../lib/csscomb');
+var assert = require('assert');
+
+describe('options/element-case', function() {
+    var comb;
+
+    beforeEach(function() {
+        comb = new Comb();
+    });
+
+    it('Invalid String should not change case of elements', function() {
+        comb.configure({ 'element-case': 'foobar' });
+        assert.equal(
+            comb.processString(
+                'LI a { color : red }'
+            ),
+            'LI a { color : red }'
+        );
+    });
+
+    it('Should switch tag name to upper case', function() {
+        comb.configure({ 'element-case': 'upper' });
+        assert.equal(
+            comb.processString(
+                'div { color: #fff }'
+            ),
+            'DIV { color: #fff }'
+        );
+    });
+
+    it('Should switch tag name to lower case', function() {
+        comb.configure({ 'element-case': 'lower' });
+        assert.equal(
+            comb.processString(
+                'DIV { color: #FFF }'
+            ),
+            'div { color: #FFF }'
+        );
+    });
+
+    it('Should switch element-case in complex rules', function() {
+        comb.configure({ 'element-case': 'lower' });
+        assert.equal(
+            comb.processString(
+                'UL > LI > .foo:not(A) { color: red }'
+            ),
+            'ul > li > .foo:not(a) { color: red }'
+        );
+    });
+
+});

test/integral.expect.css

@@ -135,7 +135,7 @@ div p em
     border-bottom: 1px solid red; /* trololo */ /* trololo */
 }
 
-a
+a:not(b)
 {
     top: 0;/* ololo */
 

test/integral.origin.css

@@ -110,14 +110,14 @@ div {
     font-size:1px;
       top:0
   }
-    div p em {
+    div P EM {
         /* upline comment*/
         font-style:italic;
 
         border-bottom:1px solid red /* trololo */ /* trololo */
     }
 
-    a{
+    a:not(B){
 top: 0;/* ololo */margin :0;}
     b
         {