Core Component and Tooltip event handling tweaks

packages/ts/src/components/tooltip/config.ts

@@ -33,7 +33,7 @@ export interface TooltipConfigInterface {
    * ```
    */
   triggers?: {
-    [selector: string]: (data: any, i: number, elements: (HTMLElement | SVGElement)[]) => string | HTMLElement | undefined | null;
+    [selector: string]: ((data: any, i: number, elements: (HTMLElement | SVGElement)[]) => string | HTMLElement | undefined | null) | undefined | null;
   };
   /** Custom DOM attributes for the tooltip. Useful when you need to refer to a specific tooltip instance
    * by using a CSS selector. Attributes configuration object has the following structure:

packages/ts/src/components/tooltip/index.ts

@@ -163,7 +163,7 @@ export class Tooltip {
   private _setContainerPosition (): void {
     // Tooltip position calculation relies on the parent position
     // If it's not set (static), we set it to `relative` (not a good practice)
-    if (getComputedStyle(this._container)?.position === 'static') {
+    if (this._container !== document.body && getComputedStyle(this._container)?.position === 'static') {
       this._container.style.position = 'relative'
     }
   }
@@ -185,6 +185,8 @@ export class Tooltip {
           // Go through all of the configured triggers
           for (const className of Object.keys(triggers)) {
             const template = triggers[className]
+            if (!template) continue // Skip if the trigger is not configured
+
             const els = selection.selectAll<HTMLElement | SVGGElement, unknown>(`.${className}`).nodes()
 
             // Go through all of the elements in the event path (from the deepest element upwards)

packages/ts/src/core/component/index.ts

@@ -141,7 +141,7 @@ export class ComponentCore<
           const els = selection.nodes()
           const i = els.indexOf(event.currentTarget as SVGGElement | HTMLElement)
           const eventFunction = events[className][eventType as VisEventType]
-          return eventFunction(d, event, i, els)
+          return eventFunction?.(d, event, i, els)
         })
       })
     })