From 6bd6b9f1ebb260eaa2670dfb45bef3702dbc76ed Mon Sep 17 00:00:00 2001 From: Kristiyan Kostadinov Date: Fri, 29 Sep 2017 22:12:25 +0200 Subject: [PATCH] fix(autocomplete): remove invalid aria markup (#7107) Removes the `aria-multiline` attribute, because it is not allowed on a `role="combobox"`. Fixes #7100. --- src/lib/autocomplete/autocomplete-trigger.ts | 1 - src/lib/autocomplete/autocomplete.spec.ts | 5 ----- 2 files changed, 6 deletions(-) diff --git a/src/lib/autocomplete/autocomplete-trigger.ts b/src/lib/autocomplete/autocomplete-trigger.ts index 15cfe649a84..abbc2c0fbda 100644 --- a/src/lib/autocomplete/autocomplete-trigger.ts +++ b/src/lib/autocomplete/autocomplete-trigger.ts @@ -99,7 +99,6 @@ export function getMatAutocompleteMissingPanelError(): Error { 'role': 'combobox', 'autocomplete': 'off', 'aria-autocomplete': 'list', - 'aria-multiline': 'false', '[attr.aria-activedescendant]': 'activeOption?.id', '[attr.aria-expanded]': 'panelOpen.toString()', '[attr.aria-owns]': 'autocomplete?.id', diff --git a/src/lib/autocomplete/autocomplete.spec.ts b/src/lib/autocomplete/autocomplete.spec.ts index 3096f21e8fc..9957cf4b03b 100644 --- a/src/lib/autocomplete/autocomplete.spec.ts +++ b/src/lib/autocomplete/autocomplete.spec.ts @@ -1154,11 +1154,6 @@ describe('MatAutocomplete', () => { .toEqual('list', 'Expected aria-autocomplete attribute to equal list.'); }); - it('should set aria-multiline to false', () => { - expect(input.getAttribute('aria-multiline')) - .toEqual('false', 'Expected aria-multiline attribute to equal false.'); - }); - it('should set aria-activedescendant based on the active option', async(() => { fixture.componentInstance.trigger.openPanel(); fixture.detectChanges();