45 lines
1.4 KiB
JavaScript
45 lines
1.4 KiB
JavaScript
|
"use strict";
|
||
|
|
||
|
var _jsxAstUtils = require("jsx-ast-utils");
|
||
|
|
||
|
var _schemas = require("../util/schemas");
|
||
|
|
||
|
/**
|
||
|
* @fileoverview Enforce usage of onBlur over onChange for accessibility.
|
||
|
* @author Ethan Cohen
|
||
|
*/
|
||
|
// ----------------------------------------------------------------------------
|
||
|
// Rule Definition
|
||
|
// ----------------------------------------------------------------------------
|
||
|
var errorMessage = 'onBlur must be used instead of onchange, unless absolutely necessary and it causes no negative consequences for keyboard only or screen reader users.';
|
||
|
var applicableTypes = ['select', 'option'];
|
||
|
var schema = (0, _schemas.generateObjSchema)();
|
||
|
module.exports = {
|
||
|
meta: {
|
||
|
docs: {
|
||
|
url: 'https://github.com/evcohen/eslint-plugin-jsx-a11y/tree/master/docs/rules/no-onchange.md'
|
||
|
},
|
||
|
schema: [schema]
|
||
|
},
|
||
|
create: function create(context) {
|
||
|
return {
|
||
|
JSXOpeningElement: function JSXOpeningElement(node) {
|
||
|
var nodeType = (0, _jsxAstUtils.elementType)(node);
|
||
|
|
||
|
if (applicableTypes.indexOf(nodeType) === -1) {
|
||
|
return;
|
||
|
}
|
||
|
|
||
|
var onChange = (0, _jsxAstUtils.getProp)(node.attributes, 'onChange');
|
||
|
var hasOnBlur = (0, _jsxAstUtils.getProp)(node.attributes, 'onBlur') !== undefined;
|
||
|
|
||
|
if (onChange && !hasOnBlur) {
|
||
|
context.report({
|
||
|
node,
|
||
|
message: errorMessage
|
||
|
});
|
||
|
}
|
||
|
}
|
||
|
};
|
||
|
}
|
||
|
};
|