Skip to content

Commit

Permalink
Dropdown: Removing aria-required from multiselect scenarios since it'…
Browse files Browse the repository at this point in the history
…s not allowed for role='button'.
  • Loading branch information
khmakoto committed Aug 5, 2020
1 parent c7ffce2 commit 4916bfe
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -269,6 +269,7 @@ export class DropdownBase extends React.Component<IDropdownInternalProps, IDropd
{
role: 'listbox',
childRole: 'option',
ariaRequired: required,
ariaSetSize: this._sizePosCache.optionSetSize,
ariaPosInSet: this._sizePosCache.positionInSet(selectedIndices[0]),
ariaSelected: selectedIndices[0] === undefined ? undefined : true,
Expand Down Expand Up @@ -313,7 +314,7 @@ export class DropdownBase extends React.Component<IDropdownInternalProps, IDropd
hasErrorMessage ? this._id + '-errorMessage' : undefined,
)}
aria-activedescendant={ariaActiveDescendant}
aria-required={required}
aria-required={ariaAttrs.ariaRequired}
aria-disabled={disabled}
aria-owns={isOpen ? this._listId : undefined}
{...divProps}
Expand Down

0 comments on commit 4916bfe

Please sign in to comment.