From 08509816b2d1c3dc3d68334f4f7bbadf342057c6 Mon Sep 17 00:00:00 2001 From: Kristiyan Kostadinov Date: Sat, 5 Aug 2017 01:09:42 +0200 Subject: [PATCH] fix(input): required asterisk being read out by screen readers (#6277) Fixes the asterisk that gets added after a required input being read out by screen readers. --- src/lib/input/input-container.html | 5 ++++- src/lib/input/input-container.spec.ts | 9 +++++++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/src/lib/input/input-container.html b/src/lib/input/input-container.html index 05b9ebabb24..3e2de133f9c 100644 --- a/src/lib/input/input-container.html +++ b/src/lib/input/input-container.html @@ -17,7 +17,10 @@ *ngIf="_hasPlaceholder()"> {{_mdInputChild.placeholder}} - * + diff --git a/src/lib/input/input-container.spec.ts b/src/lib/input/input-container.spec.ts index cbc515d42b6..4c1b92ed567 100644 --- a/src/lib/input/input-container.spec.ts +++ b/src/lib/input/input-container.spec.ts @@ -377,6 +377,15 @@ describe('MdInputContainer without forms', function () { expect(el.nativeElement.textContent).toMatch(/hello\s+\*/g); }); + it('should hide the required star from screen readers', () => { + let fixture = TestBed.createComponent(MdInputContainerPlaceholderRequiredTestComponent); + fixture.detectChanges(); + + let el = fixture.debugElement.query(By.css('.mat-placeholder-required')).nativeElement; + + expect(el.getAttribute('aria-hidden')).toBe('true'); + }); + it('hide placeholder required star when set to hide the required marker', () => { let fixture = TestBed.createComponent(MdInputContainerPlaceholderRequiredTestComponent); fixture.detectChanges();