Merge branch 'alt-text' of https://github.com/jwyung/eslint-plugin-jsx-a11y into jwyung-alt-text

Author: jessebeach

__tests__/src/rules/alt-text-test.js

@@ -83,6 +83,8 @@ ruleTester.run('alt-text', rule, {
     { code: '<img alt={error ? "not working": "working"} />' },
     { code: '<img alt={undefined ? "working": "not working"} />' },
     { code: '<img alt={plugin.name + " Logo"} />' },
+    { code: '<img aria-label="foo" />' },
+    { code: '<img aria-labelledby="id1" />' },
 
     // DEFAULT <object> TESTS
     { code: '<object aria-label="foo" />' },

src/rules/alt-text.js

@@ -34,8 +34,16 @@ const schema = generateObjSchema({
 
 const ruleByElement = {
   img(context, node) {
-    const nodeType = elementType(node);
+    // Check for label props (`aria-label` and `aria-labelledby`) to provide text alternative
+    const ariaLabelProp = getProp(node.attributes, 'aria-label');
+    const arialLabelledByProp = getProp(node.attributes, 'aria-labelledby');
+    const hasLabel = ariaLabelProp !== undefined || arialLabelledByProp !== undefined;
 
+    if (hasLabel) {
+      return;
+    }
+
+    const nodeType = elementType(node);
     const altProp = getProp(node.attributes, 'alt');
 
     // Missing alt prop error.