fix: set aria-checked properly when checked is null (#164)

vaadin · Mar 18, 2020 · f8ebd2c · f8ebd2c
1 parent 0b3b8e8
commit f8ebd2c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/vaadin-checkbox.html b/src/vaadin-checkbox.html
@@ -222,7 +222,7 @@
           if (this.indeterminate) {
             this.setAttribute('aria-checked', 'mixed');
           } else {
-            this.setAttribute('aria-checked', checked);
+            this.setAttribute('aria-checked', Boolean(checked));
           }
         }
 

diff --git a/test/accessibility.html b/test/accessibility.html
@@ -70,6 +70,10 @@
           expect(vaadinCheckbox.getAttribute('aria-checked')).to.be.eql('mixed');
         });
 
+        it('should set aria-checked to "false" when checked is set to null', () => {
+          vaadinCheckbox.checked = null;
+          expect(vaadinCheckbox.getAttribute('aria-checked')).to.eql('false');
+        });
       });
 
     });