Skip to content

Commit

Permalink
Only Add aria-controls to multi-select trigger if select is open
Browse files Browse the repository at this point in the history
Otherwise, axe complains that the aria-controls element it refers to doesn't exist.
  • Loading branch information
Francesco Novy authored and mydea committed May 18, 2020
1 parent 5515ca2 commit 096acd1
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
aria-labelledby={{@ariaLabelledBy}}
aria-label={{@ariaLabel}}
value={{@select.searchText}}
aria-controls={{@listboxId}}
aria-controls={{if @select.isOpen @listboxId}}
style={{this.triggerMultipleInputStyle}}
placeholder={{this.maybePlaceholder}}
disabled={{@select.disabled}}
Expand Down

0 comments on commit 096acd1

Please sign in to comment.