fix: remove check for unsupported type

vaadin · May 29, 2020 · 88810d0 · 88810d0
1 parent 6f99039
commit 88810d0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/src/vaadin-radio-group.html b/src/vaadin-radio-group.html
@@ -502,17 +502,17 @@
 
         /** @private */
         _labelChanged(label) {
-          this._setOrToggleAttribute('has-label', label === 0 || !!label);
+          this._setOrToggleAttribute('has-label', !!label);
         }
 
         /** @private */
         _errorMessageChanged(errorMessage) {
-          this._setOrToggleAttribute('has-error-message', errorMessage === 0 || !!errorMessage);
+          this._setOrToggleAttribute('has-error-message', !!errorMessage);
         }
 
         /** @private */
         _helperTextChanged(helperText) {
-          this._setOrToggleAttribute('has-helper', helperText === 0 || !!helperText);
+          this._setOrToggleAttribute('has-helper', !!helperText);
         }
 
         /** @private */
@@ -536,7 +536,7 @@
           // The filter is added due to shady DOM triggering this slotchange event on helperText prop change
           this._hasSlottedHelper = slottedNodes.filter(node => node.nodeType !== 3).length > 0;
 
-          this._setOrToggleAttribute('has-helper', this._hasSlottedHelper ? 'slotted' : this.helperText === 0 || !!this.helperText);
+          this._setOrToggleAttribute('has-helper', this._hasSlottedHelper ? 'slotted' : !!this.helperText);
         }
 
         /** @private */

diff --git a/test/vaadin-radio-group.html b/test/vaadin-radio-group.html
@@ -467,11 +467,6 @@
           expect(vaadinRadioButtonGroup.hasAttribute('has-helper')).to.be.false;
         });
 
-        it('setting number helper updates has-helper attribute', function() {
-          vaadinRadioButtonGroup.helperText = 0;
-          expect(vaadinRadioButtonGroup.hasAttribute('has-helper')).to.be.true;
-        });
-
         it('field with slotted helper updates has-helper attribute', function() {
           const radioButtonGroupWithSlottedHelper = fixture('default-with-slotted-helper');
           radioButtonGroupWithSlottedHelper._observer.flush();