fix(material/checkbox): add aria-checked (#27970)

* fix(material/checkbox): add aria-checked * fix(material/checkbox): move aria-checked test (cherry picked from commit cc2f2c5)
angular · Oct 23, 2023 · 215b92a · 215b92a
1 parent d74e508
commit 215b92a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/material/checkbox/checkbox.html b/src/material/checkbox/checkbox.html
@@ -11,6 +11,7 @@
            [attr.aria-label]="ariaLabel || null"
            [attr.aria-labelledby]="ariaLabelledby"
            [attr.aria-describedby]="ariaDescribedby"
+           [attr.aria-checked]="indeterminate ? 'mixed' : null"
            [attr.name]="name"
            [attr.value]="value"
            [checked]="checked"

diff --git a/src/material/checkbox/checkbox.spec.ts b/src/material/checkbox/checkbox.spec.ts
@@ -55,6 +55,9 @@ describe('MDC-based MatCheckbox', () => {
 
       expect(checkboxInstance.checked).toBe(true);
       expect(inputElement.checked).toBe(true);
+      expect(inputElement.hasAttribute('aria-checked'))
+        .withContext('Expect aria-checked attribute to not be used')
+        .toBe(false);
 
       testComponent.isChecked = false;
       fixture.detectChanges();
@@ -103,9 +106,6 @@ describe('MDC-based MatCheckbox', () => {
 
       expect(inputElement.checked).toBe(false);
       expect(inputElement.indeterminate).toBe(true);
-      expect(inputElement.hasAttribute('aria-checked'))
-        .withContext('Expect aria-checked attribute to not be used')
-        .toBe(false);
 
       testComponent.isIndeterminate = false;
       fixture.detectChanges();
@@ -145,9 +145,6 @@ describe('MDC-based MatCheckbox', () => {
       expect(inputElement.indeterminate).toBe(true);
       expect(inputElement.checked).toBe(true);
       expect(testComponent.isIndeterminate).toBe(true);
-      expect(inputElement.hasAttribute('aria-checked'))
-        .withContext('Expect aria-checked attribute to not be used')
-        .toBe(false);
 
       inputElement.click();
       fixture.detectChanges();