Merge pull request #1 from hrobertking/transparent-color

Adding transparent-color rule

Author: hrobertking

src/rules/transparent-color.js

@@ -0,0 +1,37 @@
+/*
+ * Rule: Set color to a visible value
+ *
+ */
+
+CSSLint.addRule({
+
+    // rule information
+    id: "transparent-color",
+    name: "Named value 'transparent' used for color",
+    desc: "Checks for a value for color that is transparent",
+    browsers: "All",
+
+    // initialization
+    init: function(parser, reporter) {
+        "use strict";
+        var rule = this;
+
+        // check for use of "transparent" as a value for "color"
+        parser.addListener("property", function(event) {
+            var name = event.property.toString().toLowerCase(),
+                parts = event.value.parts,
+                ndx = parts.length - 1;
+
+            if (name === "color") {
+                while (ndx > -1) {
+                    if (parts[ndx].type === "identifier" && parts[ndx].value === "transparent") {
+                        reporter.report("Color cannot be 'transparent'.", event.line, event.col, rule);
+                        break;
+                    }
+                    ndx -= 1;
+                }
+            }
+        });
+    }
+
+});

tests/rules/transparent-color.js

@@ -0,0 +1,16 @@
+(function() {
+    "use strict";
+    var Assert = YUITest.Assert;
+
+    YUITest.TestRunner.add(new YUITest.TestCase({
+
+        name: "transparent color Errors",
+
+        "A 'transparent' value for color should result in a warning": function() { 
+            var result = CSSLint.verify("h1 { color: transparent; }", { "transparent-color": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Color cannot be 'transparent'.", result.messages[0].message);
+        }
+    }));
+})();