fix: EventDispatcher allow events without payload

src/components/drawer/bl-drawer.ts

@@ -111,21 +111,21 @@ export default class BlDrawer extends LitElement {
         clearTimeout(this.domExistenceSchedule);
       }
       this.domExistence = true;
+
       window.setTimeout(() => {
         if (this.embedUrl && this._drawerIframe) {
           this._drawerIframe.src = this.embedUrl;
         }
       });
       // FIXME: Allow events without payload
-      this.onOpen("");
+      this.onOpen();
     } else {
       // Give some time for exit animation
       this.domExistenceSchedule = window.setTimeout(() => {
         this.domExistence = false;
       }, 1000);
 
-      // FIXME: Allow events without payload
-      this.onClose("");
+      this.onClose();
     }
   }
 

src/components/popover/bl-popover.ts

@@ -195,7 +195,7 @@ export default class BlPopover extends LitElement {
   show() {
     this._visible = true;
     this.setPopover();
-    this.onBlPopoverShow("");
+    this.onBlPopoverShow();
     document.addEventListener("click", this._handleClickOutside);
     document.addEventListener("keydown", this._handleKeydownEvent);
     document.addEventListener("bl-popover-show", this._handlePopoverShowEvent);
@@ -209,7 +209,7 @@ export default class BlPopover extends LitElement {
     document.removeEventListener("click", this._handleClickOutside);
     document.removeEventListener("keydown", this._handleKeydownEvent);
     document.removeEventListener("bl-popover-show", this._handlePopoverShowEvent);
-    this.onBlPopoverHide("");
+    this.onBlPopoverHide();
   }
 
   /**

src/components/tooltip/bl-tooltip.test.ts

@@ -144,7 +144,7 @@ describe("bl-tooltip", () => {
     const ev = await oneEvent(el, "bl-tooltip-show");
 
     expect(ev).to.exist;
-    expect(ev.detail).to.be.equal("");
+    expect(ev.detail).to.be.null;
   });
 
   it("should fires bl-tooltip-hide on mouse leave", async () => {
@@ -169,7 +169,7 @@ describe("bl-tooltip", () => {
     const ev = await oneEvent(el, "bl-tooltip-hide");
 
     expect(ev).to.exist;
-    expect(ev.detail).to.be.equal("");
+    expect(ev.detail).to.be.null;
   });
 
   it("should show/hide with focus and blur events", async () => {

src/components/tooltip/bl-tooltip.ts

@@ -97,17 +97,19 @@
    * Shows tooltip
    */
   show() {
-    this._popover.target = this.target ?? this.trigger;
-    this._popover.show();
-    this.onShow("");
+
+  this._popover.target = this.target ?? this.trigger;
+  this._popover.show();
+  this.onShow();
   }
 
   /**
    * Hides tooltip
    */
   hide() {
-    this._popover.hide();
-    this.onHide("");
+
+  this._popover.hide();
+  this.onHide();
   }
 
   /**
@@ -136,7 +138,7 @@
       <bl-popover
         .target="${this.target ?? this.trigger}"
         placement="${ifDefined(this.placement)}"
-        @bl-popover-hide="${() => this.onHide("")}"
+        @bl-popover-hide="${() => this.onHide()}"
       >
         <slot class="content" id="tooltip" role="tooltip"></slot>
       </bl-popover>

src/utilities/event.ts

@@ -5,11 +5,11 @@ export interface EventOptions {
 }
 
 export interface EventDispatcher<T> {
-  (value: T, options?: EventOptions): CustomEvent<T>;
+  (value?: T, options?: EventOptions): CustomEvent<T>;
 }
 
 function dispatcher<T>(target: HTMLElement, eventName: string): EventDispatcher<T> {
-  return function (value: T, options?: EventOptions) {
+  return function (value?: T, options?: EventOptions) {
     const customEvent = new CustomEvent<T>(eventName, {
       detail: value,
       bubbles: true,