From 426970ca005a874ee5d1cebc4e5c4bee3385de4c Mon Sep 17 00:00:00 2001 From: crisbeto Date: Fri, 15 Sep 2017 19:29:03 +0200 Subject: [PATCH] fix(autocomplete): remove invalid aria markup 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 972b78fe9823..fda3c3886cb1 100644 --- a/src/lib/autocomplete/autocomplete-trigger.ts +++ b/src/lib/autocomplete/autocomplete-trigger.ts @@ -100,7 +100,6 @@ export function getMdAutocompleteMissingPanelError(): 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 308382a9c0f9..647673891a0f 100644 --- a/src/lib/autocomplete/autocomplete.spec.ts +++ b/src/lib/autocomplete/autocomplete.spec.ts @@ -1104,11 +1104,6 @@ describe('MdAutocomplete', () => { .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();