ariaActivedescendant = false not valid aria

Fixes alphagov#434
Jordan-Hall · Apr 21, 2022 · b9eb9c3 · b9eb9c3
1 parent d892838
commit b9eb9c3
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/autocomplete.js b/src/autocomplete.js
@@ -457,6 +457,10 @@ export default class Autocomplete extends Component {
       'aria-describedby': assistiveHintID
     } : null
 
+    const ariaActivedescendantProp = (optionFocused) ? {
+      'aria-activedescendant': `${id}__option--${focused}`
+    } : null
+
     let dropdownArrow
 
     // we only need a dropdown arrow if showAllValues is set to a truthy value
@@ -492,7 +496,7 @@ export default class Autocomplete extends Component {
 
         <input
           aria-expanded={menuOpen ? 'true' : 'false'}
-          aria-activedescendant={optionFocused ? `${id}__option--${focused}` : false}
+          {...ariaActivedescendantProp}
           aria-owns={`${id}__listbox`}
           aria-autocomplete={(this.hasAutoselect()) ? 'both' : 'list'}
           {...ariaDescribedProp}