@lion/ui 0.18.0 → 0.18.1

package/components/form-core/src/choice-group/ChoiceInputMixin.js

@@ -238,7 +238,7 @@ const ChoiceInputMixinImplementation = superclass =>
      */
     // eslint-disable-next-line no-unused-vars
     _toggleChecked(ev) {
-      if (this.disabled) {
+      if (this.disabled || this.readOnly) {
         return;
       }
       this._isHandlingUserInput = true;

package/components/switch/src/LionSwitch.js

@@ -148,7 +148,7 @@ export class LionSwitch extends ScopedElementsMixin(ChoiceInputMixin(LionField))
    * @protected
    */
   _onLabelClick() {
-    if (this.disabled) {
+    if (this.disabled || this.readOnly) {
       return;
     }
     this._inputNode.focus();

package/components/switch/test/lion-switch.test.js

@@ -72,6 +72,13 @@ describe('lion-switch', () => {
     expect(isActiveElement(_inputNode)).to.be.false;
   });
 
+  it('clicking the label should not toggle the checked state when readonly', async () => {
+    const el = await fixture(html`<lion-switch readonly label="Enable Setting"></lion-switch>`);
+    const { _labelNode } = getFormControlMembers(el);
+    _labelNode.click();
+    expect(el.checked).to.be.false;
+  });
+
   it('should sync its "disabled" state to child button', async () => {
     const el = await fixture(html`<lion-switch disabled></lion-switch>`);
     const { _inputNode } = getSwitchMembers(el);

package/custom-elements.json

@@ -2166,6 +2166,36 @@
         }
       ]
     },
+    {
+      "kind": "javascript-module",
+      "path": "exports/define-helpers/sb-action-logger.js",
+      "declarations": [],
+      "exports": [
+        {
+          "kind": "custom-element-definition",
+          "name": "sb-action-logger",
+          "declaration": {
+            "name": "SbActionLogger",
+            "module": "/exports/helpers.js"
+          }
+        }
+      ]
+    },
+    {
+      "kind": "javascript-module",
+      "path": "exports/define-helpers/sb-locale-switcher.js",
+      "declarations": [],
+      "exports": [
+        {
+          "kind": "custom-element-definition",
+          "name": "sb-locale-switcher",
+          "declaration": {
+            "name": "SbLocaleSwitcher",
+            "module": "/exports/helpers.js"
+          }
+        }
+      ]
+    },
     {
       "kind": "javascript-module",
       "path": "exports/define/lion-accordion.js",
@@ -2871,36 +2901,6 @@
         }
       ]
     },
-    {
-      "kind": "javascript-module",
-      "path": "exports/define-helpers/sb-action-logger.js",
-      "declarations": [],
-      "exports": [
-        {
-          "kind": "custom-element-definition",
-          "name": "sb-action-logger",
-          "declaration": {
-            "name": "SbActionLogger",
-            "module": "/exports/helpers.js"
-          }
-        }
-      ]
-    },
-    {
-      "kind": "javascript-module",
-      "path": "exports/define-helpers/sb-locale-switcher.js",
-      "declarations": [],
-      "exports": [
-        {
-          "kind": "custom-element-definition",
-          "name": "sb-locale-switcher",
-          "declaration": {
-            "name": "SbLocaleSwitcher",
-            "module": "/exports/helpers.js"
-          }
-        }
-      ]
-    },
     {
       "kind": "javascript-module",
       "path": "components/accordion/src/LionAccordion.js",