fix: reflect readonly property to attribute (#134)

vaadin · Mar 18, 2020 · 28af3ba · 28af3ba
1 parent 3492426
commit 28af3ba
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/vaadin-time-picker.html b/src/vaadin-time-picker.html
@@ -218,7 +218,8 @@
              */
             readonly: {
               type: Boolean,
-              value: false
+              value: false,
+              reflectToAttribute: true
             },
 
             /**

diff --git a/test/input-test.html b/test/input-test.html
@@ -205,6 +205,11 @@
           });
         });
 
+        it('should reflect to attribute when readonly property is set', () => {
+          timePicker.readonly = true;
+          expect(timePicker.hasAttribute('readonly')).to.be.true;
+        });
+
         describe('aria', () => {
           it('text-field should have the `aria-label` attribute', () => {
             expect(timePicker.__inputElement.hasAttribute('aria-label')).to.be.false;