diff --git a/rules/react-a11y.js b/rules/react-a11y.js index 9e1c664..ba07d18 100644 --- a/rules/react-a11y.js +++ b/rules/react-a11y.js @@ -3,9 +3,6 @@ module.exports = { 'jsx-a11y', 'react' ], - ecmaFeatures: { - jsx: true - }, rules: { // Enforce that anchors have content // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/anchor-has-content.md @@ -31,11 +28,11 @@ module.exports = { // disallow href "#" // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/href-no-hash.md - 'jsx-a11y/href-no-hash': [2, ['a']], + //'jsx-a11y/href-no-hash': [2, ['a']], // Require to have a non-empty `alt` prop, or role="presentation" // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/img-has-alt.md - 'jsx-a11y/img-has-alt': 2, + //'jsx-a11y/img-has-alt': 2, // Prevent img alt text from containing redundant words like "image", "picture", or "photo" // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/img-redundant-alt.md @@ -43,7 +40,7 @@ module.exports = { // require that JSX labels use "htmlFor" // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/label-has-for.md - 'jsx-a11y/label-has-for': [2, ['label']], + 'jsx-a11y/label-has-for': [2, { "components": ["label"] }], // require that mouseover/out come with focus/blur, for keyboard-only users // TODO: evaluate @@ -84,7 +81,7 @@ module.exports = { // ensure tags have content and are not aria-hidden // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/heading-has-content.md - 'jsx-a11y/heading-has-content': [2, ['']], + 'jsx-a11y/heading-has-content': [2, { "components": [''] }], // require HTML elements to have a "lang" prop // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/html-has-lang.md @@ -96,7 +93,7 @@ module.exports = { // prevent marquee elements // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/no-marquee.md - 'jsx-a11y/no-marquee': 2, + //'jsx-a11y/no-marquee': 2, // only allow to have the "scope" attr // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/scope.md diff --git a/rules/react.js b/rules/react.js index 53961c3..853b9d3 100644 --- a/rules/react.js +++ b/rules/react.js @@ -7,9 +7,6 @@ module.exports = { jsx: true, }, }, - ecmaFeatures: { - jsx: true - }, // View link below for react rules documentation // https://github.com/yannickcr/eslint-plugin-react#list-of-supported-rules @@ -170,7 +167,7 @@ module.exports = { // Restrict file extensions that may be required // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/require-extension.md - 'react/require-extension': [2, { extensions: ['.jsx', '.js'] }], + //'react/require-extension': [2, { extensions: ['.jsx', '.js'] }], // Require render() methods to return something // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/require-render-return.md @@ -181,8 +178,12 @@ module.exports = { 'react/self-closing-comp': 2, // Enforce spaces before the closing bracket of self-closing JSX elements - // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-space-before-closing.md - 'react/jsx-space-before-closing': [2, 'always'], + // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-tag-spacing.md + 'react/jsx-tag-spacing': [2, { + closingSlash: "never", + beforeSelfClosing: "always", + afterOpening: "never" + }], // Enforce component methods order // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/sort-comp.md