@marianmeres/stuic 1.18.0 → 1.20.0

package/dist/actions/tooltip/_set-position.js

@@ -40,8 +40,6 @@ parent, div, arrow, opts, log) => {
             // position the actual tooltip/popover
             div.style.left = `${r.position[safe].x}px`;
             div.style.top = `${r.position[safe].y}px`;
-            // div.style.left = `${0}px`;
-            // div.style.top = `${0}px`;
             // now dance with the arrow...
             let arrowStyles = {
                 borderStyle: 'solid',

package/dist/actions/tooltip/tooltip.d.ts

@@ -16,6 +16,7 @@ export declare class PopoverConfig {
     static defaultOptions: Partial<TooltipOptions>;
 }
 export type TooltipLogger = (...args: any[]) => void;
+export type TooltipTrigger = 'hover' | 'focus' | 'click';
 export interface TooltipOptions {
     content: string;
     popover: HTMLElement | null;
@@ -24,7 +25,7 @@ export interface TooltipOptions {
     delay: number;
     class: string;
     arrowClass: string;
-    triggers: string[];
+    triggers: TooltipTrigger[];
     logger?: TooltipLogger;
     boundaryRoot?: HTMLElement;
     arrowSize: number;

package/dist/actions/tooltip/tooltip.js

@@ -144,7 +144,7 @@ export function tooltip(node, initialOptions = {}) {
             _resetDelayTimer();
         }, _delay);
     };
-    // use popover if provided, otherwise new div will be createed
+    // use popover if provided, otherwise new div will be created
     let div = opts.popover;
     let arrow;
     let _isOn = writable(false); // internal state store
@@ -168,14 +168,14 @@ export function tooltip(node, initialOptions = {}) {
         }
         // measure stuff and set position (provided opts.alignment is considered just as
         // "preferred", which means it may be overwritten if there is no available space)
-        if (!(await _setPosition(opts.boundaryRoot, node, div, arrow, opts, _log))) {
-            _makeInVisible(div, arrow, _log);
-        }
-        else {
+        if (await _setPosition(opts.boundaryRoot, node, div, arrow, opts, _log)) {
             // finally, fade in
             _makeVisible(div, arrow, _log);
             setTimeout(() => _isOn.set(true), _TRANSITION_OPACITY_DUR);
         }
+        else {
+            _makeInVisible(div, arrow, _log);
+        }
     };
     let show = () => _planDelayedExec(_show, opts.delay);
     //
@@ -195,6 +195,9 @@ export function tooltip(node, initialOptions = {}) {
     };
     //
     const destroy = () => {
+        // for obscure cases, where parent node might have be removed from DOM
+        // before the planned show/hide happens
+        _resetDelayTimer();
         if (!div && !arrow && !opts.popover && !storeGet(_isOn))
             return;
         _log('destroy');
@@ -209,8 +212,8 @@ export function tooltip(node, initialOptions = {}) {
     //
     let unsubs = [_isOn.subscribe((v) => opts?.notifier?.set(v))];
     // by default, listen to windowSize change, as well as window and boundaryRoot scroll
-    const _boundaryRootScroll = writable(0);
-    const onScroll = () => _boundaryRootScroll.set(Date.now());
+    const _scrollSignal = writable(0);
+    const onScroll = () => _scrollSignal.set(Date.now());
     if (opts.boundaryRoot) {
         opts.boundaryRoot.addEventListener('scroll', onScroll);
         unsubs.push(() => opts.boundaryRoot?.removeEventListener('scroll', onScroll));
@@ -218,21 +221,21 @@ export function tooltip(node, initialOptions = {}) {
     // also listen to window scroll
     window.addEventListener('scroll', onScroll);
     unsubs.push(() => window.removeEventListener('scroll', onScroll));
-    const _positionTriggers = [_boundaryRootScroll, windowSize];
+    const _setPositionTriggers = [_scrollSignal, windowSize];
     if (opts.touch?.subscribe)
-        _positionTriggers.push(opts.touch);
-    const touch = derived(_positionTriggers, ([_]) => Date.now());
+        _setPositionTriggers.push(opts.touch);
+    const touch = derived(_setPositionTriggers, ([_]) => Date.now());
     // final, derived, notifier
     let _touchCount = 0;
     unsubs.push(touch.subscribe(async () => {
         // ignore first
         if (_touchCount++) {
             _log('touch...');
-            if (!(await _setPosition(opts.boundaryRoot, node, div, arrow, opts, _log))) {
-                _makeInVisible(div, arrow, _log);
+            if (await _setPosition(opts.boundaryRoot, node, div, arrow, opts, _log)) {
+                _makeVisible(div, arrow, _log);
             }
             else {
-                _makeVisible(div, arrow, _log);
+                _makeInVisible(div, arrow, _log);
             }
         }
     }));

package/dist/index.d.ts

@@ -6,7 +6,7 @@ export { default as LocalColorScheme } from './components/ColorScheme/LocalColor
 export { ColorScheme } from './components/ColorScheme/color-scheme.js';
 export { default as Drawer, createDrawerStore } from './components/Drawer/Drawer.svelte';
 export { default as HoverExpandableWidth } from './components/HoverExpandableWidth/HoverExpandableWidth.svelte';
-export { default as Switch } from './components/Switch/Switch.svelte';
+export { default as Switch, SwitchConfig } from './components/Switch/Switch.svelte';
 export { default as X } from './components/X/X.svelte';
 export { focusTrap } from './actions/focus-trap.js';
 export { onOutside } from './actions/on-outside.js';

package/dist/index.js

@@ -14,7 +14,7 @@ export { default as Drawer, createDrawerStore } from './components/Drawer/Drawer
 //
 export { default as HoverExpandableWidth } from './components/HoverExpandableWidth/HoverExpandableWidth.svelte';
 //
-export { default as Switch } from './components/Switch/Switch.svelte';
+export { default as Switch, SwitchConfig } from './components/Switch/Switch.svelte';
 //
 export { default as X } from './components/X/X.svelte';
 // actions

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@marianmeres/stuic",
-	"version": "1.18.0",
+	"version": "1.20.0",
 	"scripts": {
 		"dev": "vite dev",
 		"build": "vite build && npm run package && node ./scripts/date.js",