Enforce no accessKey prop.

Author: Ethan Cohen

index.js

@@ -5,7 +5,8 @@ module.exports = {
     'img-uses-alt': require('./lib/rules/img-uses-alt'),
     'onClick-uses-role': require('./lib/rules/onClick-uses-role'),
     'mouseEvents-require-keyEvents': require('./lib/rules/mouseEvents-require-keyEvents'),
-    'use-onblur-not-onchange': require('./lib/rules/use-onblur-not-onchange')
+    'use-onblur-not-onchange': require('./lib/rules/use-onblur-not-onchange'),
+    'no-access-key': require('./lib/rules/no-access-key')
   },
   configs: {
     recommended: {

lib/rules/no-access-key.js

@@ -0,0 +1,33 @@
+/**
+ * @fileoverview Enforce no accesskey attribute on element.
+ * @author Ethan Cohen
+ */
+'use strict';
+
+// ------------------------------------------------------------------------------
+// Rule Definition
+// ------------------------------------------------------------------------------
+
+var hasAttribute = require('../hasAttribute');
+
+module.exports = function(context) {
+
+  return {
+    JSXOpeningElement: function(node) {
+      var hasAccessKey = hasAttribute(node.attributes, 'accesskey');
+
+      if (Boolean(hasAccessKey) === true) {
+        context.report({
+          node: node,
+          message: 'Inconsistencies between keyboard shortcuts and keyboard commands used by screenreader ' +
+          'and keyboard only users create accessibility complications so to avoid complications, access keys ' +
+          'should not be used.'
+        });
+      }
+    }
+  };
+};
+
+module.exports.schema = [{
+  type: 'object'
+}];

tests/lib/rules/mouseEvents-require-keyEvents.js

@@ -1,5 +1,5 @@
 /**
- * @fileoverview Enforce img tags use alt attribute.
+ * @fileoverview Enforce onmouseover/onmouseout are accompanied by onfocus/onblur.
  * @author Ethan Cohen
  */
 

tests/lib/rules/no-access-key.js

@@ -0,0 +1,44 @@
+/**
+ * @fileoverview Enforce no accesskey attribute on element.
+ * @author Ethan Cohen
+ */
+
+'use strict';
+
+// -----------------------------------------------------------------------------
+// Requirements
+// -----------------------------------------------------------------------------
+
+var rule = require('../../../lib/rules/no-access-key');
+var RuleTester = require('eslint').RuleTester;
+
+var parserOptions = {
+  ecmaVersion: 6,
+  ecmaFeatures: {
+    jsx: true
+  }
+};
+
+// -----------------------------------------------------------------------------
+// Tests
+// -----------------------------------------------------------------------------
+
+var ruleTester = new RuleTester();
+
+var expectedError = {
+  message: 'Inconsistencies between keyboard shortcuts and keyboard commands used by screenreader ' +
+    'and keyboard only users create accessibility complications so to avoid complications, access keys ' +
+    'should not be used.',
+  type: 'JSXOpeningElement'
+};
+
+ruleTester.run('no-access-key', rule, {
+  valid: [
+    {code: '<div />;', parserOptions: parserOptions}
+  ],
+  invalid: [
+    {code: '<div accesskey="h" />;', errors: [expectedError], parserOptions: parserOptions},
+    {code: '<div accessKey="h" />', errors: [expectedError], parserOptions: parserOptions},
+    {code: '<section acCesSKeY="y" />', errors: [expectedError], parserOptions: parserOptions}
+  ]
+});

tests/lib/rules/onClick-uses-role.js

@@ -1,5 +1,5 @@
 /**
- * @fileoverview Enforce img tags use alt attribute.
+ * @fileoverview Enforce non-interactive elements with click handlers use role attribute.
  * @author Ethan Cohen
  */
 

tests/lib/rules/use-onblur-not-onchange.js

@@ -1,5 +1,5 @@
 /**
- * @fileoverview Enforce img tags use alt attribute.
+ * @fileoverview Enforce usage of onBlur over onChange for accessibility.
  * @author Ethan Cohen
  */