Consider selectors containg #id or .class qualified

Author: gurdiga

src/rules/unqualified-attributes.js

@@ -13,11 +13,13 @@ CSSLint.addRule({
     //initialization
     init: function(parser, reporter) {
         "use strict";
+
         var rule = this;
 
         parser.addListener("startrule", function(event) {
 
             var selectors = event.selectors,
+                selectorContainsClassOrId = false,
                 selector,
                 part,
                 modifier,
@@ -30,8 +32,19 @@ CSSLint.addRule({
                 if (part.type === parser.SELECTOR_PART_TYPE) {
                     for (k=0; k < part.modifiers.length; k++) {
                         modifier = part.modifiers[k];
-                        if (modifier.type === "attribute" && (!part.elementName || part.elementName === "*")) {
-                            reporter.report(rule.desc, part.line, part.col, rule);
+
+                        if (modifier.type === "class" || modifier.type === "id") {
+                          selectorContainsClassOrId = true;
+                          break;
+                        }
+                    }
+
+                    if (!selectorContainsClassOrId) {
+                        for (k=0; k < part.modifiers.length; k++) {
+                            modifier = part.modifiers[k];
+                            if (modifier.type === "attribute" && (!part.elementName || part.elementName === "*")) {
+                                reporter.report(rule.desc, part.line, part.col, rule);
+                            }
                         }
                     }
                 }

tests/rules/unqualified-attributes.js

@@ -13,6 +13,16 @@
             Assert.areEqual("Unqualified attribute selectors are known to be slow.", result.messages[0].message);
         },
 
+        "A selector that contains a class is considered qualified": function(){
+            var result = CSSLint.verify("p [href].fancy[type=text] { font-size: 10px; }", {"unqualified-attributes": 1 });
+            Assert.areEqual(0, result.messages.length);
+        },
+
+        "A selector that contains an ID is considered qualified": function(){
+            var result = CSSLint.verify("p [href]#fancy[type=text] { font-size: 10px; }", {"unqualified-attributes": 1 });
+            Assert.areEqual(0, result.messages.length);
+        },
+
         "Using an unqualified attribute selector as the right-most part should result in a warning": function() {
             var result = CSSLint.verify("p div [type=text] { font-size: 10px; }", {"unqualified-attributes": 1 });
             Assert.areEqual(1, result.messages.length);