chore(core): prevent click on disabled elements

.changeset/tasty-fishes-occur.md

@@ -0,0 +1,5 @@
+---
+'@lion/ui': patch
+---
+
+prevent click on disabled elements

packages/ui/components/core/src/DisabledMixin.js

@@ -74,6 +74,12 @@ const DisabledMixinImplementation = superclass =>
         }
       }
     }
+
+    click() {
+      if (this.disabled) return;
+
+      super.click();
+    }
   };
 
 export const DisabledMixin = dedupeMixin(DisabledMixinImplementation);

packages/ui/components/core/test/DisabledMixin.test.js

@@ -1,7 +1,8 @@
 import { expect, fixture } from '@open-wc/testing';
+import { DisabledMixin } from '@lion/ui/core.js';
 import { html } from 'lit/static-html.js';
 import { LitElement } from 'lit';
-import { DisabledMixin } from '@lion/ui/core.js';
+import sinon from 'sinon';
 
 describe('DisabledMixin', () => {
   class CanBeDisabled extends DisabledMixin(LitElement) {}
@@ -84,4 +85,24 @@ describe('DisabledMixin', () => {
     el.retractRequestToBeDisabled();
     expect(el.disabled).to.be.false;
   });
+
+  it('will not perform click when disabled', async () => {
+    const el = /** @type {CanBeDisabled} */ (
+      await fixture(html`<can-be-disabled></can-be-disabled>`)
+    );
+    const spy = sinon.spy();
+    el.addEventListener('click', spy);
+
+    el.click();
+
+    expect(spy.callCount).to.equal(1);
+
+    el.disabled = true;
+    el.click();
+    expect(spy.callCount).to.equal(1);
+
+    el.disabled = false;
+    el.click();
+    expect(spy.callCount).to.equal(2);
+  });
 });