#290 ESLint: Fix jsx-a11y/role-has-required-aria-props

mashmatrix · Mar 25, 2019 · 8830614 · 8830614
1 parent 60f4f5b
commit 8830614
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -46,7 +46,7 @@ module.exports = {
     'react/button-has-type': 2,
     'jsx-a11y/click-events-have-key-events': 1,
     'jsx-a11y/no-noninteractive-tabindex': 2,
-    'jsx-a11y/role-has-required-aria-props': 1,
+    'jsx-a11y/role-has-required-aria-props': 2,
     'jsx-a11y/anchor-is-valid': 1,
     'jsx-a11y/interactive-supports-focus': 1,
     'jsx-a11y/label-has-associated-control': 1

diff --git a/src/scripts/Lookup.js b/src/scripts/Lookup.js
@@ -424,6 +424,7 @@ class LookupCandidateList extends Component {
           className='slds-lookup__item-action react-slds-candidate'
           tabIndex={-1}
           role='option'
+          aria-selected={false}
           onKeyDown={(e) => e.keyCode === 13 && this.onSelect(entry)}
           onBlur={this.props.onBlur}
           onClick={() => this.onSelect(entry)}