Ignore function nodes in color-named

stylelint · Dec 12, 2016 · a676271 · a676271
1 parent bf60e46
commit a676271
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/lib/rules/color-named/__tests__/index.js b/lib/rules/color-named/__tests__/index.js
@@ -56,6 +56,9 @@ testRule(rule, {
   }, {
     code: "a { font: 10px/14px Brown, sans-serif; }",
     description: "ignore font family names that are colors",
+  }, {
+    code: "colors: (white: #fff);",
+    description: "ignore color names in non-standard syntax",
   } ],
 
   reject: [ {

diff --git a/lib/rules/color-named/index.js b/lib/rules/color-named/index.js
@@ -58,11 +58,23 @@ const rule = function (expectation, options) {
         return
       }
 
+      const functionNodes = []
+
       valueParser(decl.value).walk(node => {
         const value = node.value,
           type = node.type,
           sourceIndex = node.sourceIndex
 
+        if (type === "function") {
+          node.nodes.forEach((node) => {
+            functionNodes.push(node.sourceIndex)
+          })
+        }
+
+        if (functionNodes.indexOf(sourceIndex) > -1) {
+          return
+        }
+
         if (
           optionsMatches(options, "ignore", "inside-function")
           && type === "function"