uikit 3.19.5-dev.c6a7fbca6 → 3.19.5-dev.c70b3862f

package/package.json

@@ -2,7 +2,7 @@
   "name": "uikit",
   "title": "UIkit",
   "description": "UIkit is a lightweight and modular front-end framework for developing fast and powerful web interfaces.",
-  "version": "3.19.5-dev.c6a7fbca6",
+  "version": "3.19.5-dev.c70b3862f",
   "main": "dist/js/uikit.js",
   "style": "dist/css/uikit.css",
   "sideEffects": [

package/src/js/api/boot.js

@@ -1,4 +1,4 @@
-import { apply, hasAttr, inBrowser, isPlainObject, startsWith, trigger } from 'uikit-util';
+import { apply, hasAttr, inBrowser, startsWith, trigger } from 'uikit-util';
 import { components, createComponent, getComponent, getComponents } from './component';
 import { callConnected, callDisconnected } from './hooks';
 
@@ -53,16 +53,15 @@ function applyAttributeMutation({ target, attributeName }) {
     if (name) {
         if (hasAttr(target, attributeName)) {
             createComponent(name, target);
-            return;
+        } else {
+            getComponent(target, name)?.$destroy();
         }
-
-        getComponent(target, name)?.$destroy();
     }
 }
 
 function connect(node) {
     const components = getComponents(node);
-    for (const name in getComponents(node)) {
+    for (const name in components) {
         callConnected(components[name]);
     }
 
@@ -74,7 +73,7 @@ function connect(node) {
 
 function disconnect(node) {
     const components = getComponents(node);
-    for (const name in getComponents(node)) {
+    for (const name in components) {
         callDisconnected(components[name]);
     }
 }
@@ -85,5 +84,5 @@ function getComponentName(attribute) {
     }
 
     const cmp = components[attribute];
-    return cmp && (isPlainObject(cmp) ? cmp : cmp.options).name;
+    return cmp && (cmp.options || cmp).name;
 }

package/src/js/api/component.js

@@ -1,4 +1,4 @@
-import { $$, camelize, hyphenate, isEmpty, isPlainObject } from 'uikit-util';
+import { camelize, findAll, hyphenate, isEmpty, isPlainObject } from 'uikit-util';
 import App from './app';
 
 const PREFIX = 'uk-';
@@ -10,7 +10,7 @@ export function component(name, options) {
     const id = PREFIX + hyphenate(name);
 
     if (!options) {
-        if (isPlainObject(components[id])) {
+        if (!components[id].options) {
             components[id] = App.extend(components[id]);
         }
 
@@ -21,7 +21,7 @@ export function component(name, options) {
 
     App[name] = (element, data) => createComponent(name, element, data);
 
-    const opt = isPlainObject(options) ? { ...options } : options.options;
+    const opt = options.options ?? { ...options };
 
     opt.id = id;
     opt.name = name;
@@ -41,7 +41,7 @@ export function createComponent(name, element, data, ...args) {
     return Component.options.functional
         ? new Component({ data: isPlainObject(element) ? element : [element, data, ...args] })
         : element
-          ? $$(element).map(init)[0]
+          ? findAll(element).map(init)[0]
           : init();
 
     function init(element) {

package/src/js/api/computed.js

@@ -1,6 +1,6 @@
-import { hasOwn, isUndefined } from 'uikit-util';
-import { callUpdate, prependUpdate } from './update';
-import { runWatches } from './watch';
+import { hasOwn, isUndefined, observeMutation } from 'uikit-util';
+import { callUpdate, prependUpdate } from './update.js';
+import { runWatches } from './watch.js';
 
 export function initComputed(instance) {
     const { computed } = instance.$options;
@@ -13,7 +13,7 @@ export function initComputed(instance) {
         }
     }
 }
-
+const mutationOptions = { subtree: true, childList: true };
 export function registerComputed(instance, key, cb) {
     instance._hasComputed = true;
     Object.defineProperty(instance, key, {
@@ -24,6 +24,15 @@ export function registerComputed(instance, key, cb) {
 
             if (!hasOwn(_computed, key)) {
                 _computed[key] = (cb.get || cb).call(instance, $props, $el);
+                if (cb.observe && instance._computedObserver) {
+                    const selector = cb.observe.call(instance, $props);
+                    instance._computedObserver.observe(
+                        ['~', '+', '-'].includes(selector[0])
+                            ? $el.parentElement
+                            : $el.getRootNode(),
+                        mutationOptions,
+                    );
+                }
             }
 
             return _computed[key];
@@ -51,12 +60,16 @@ export function initComputedUpdates(instance) {
         events: ['resize', 'computed'],
     });
 
-    registerComputedObserver();
-    instances.add(instance);
+    instance._computedObserver = observeMutation(
+        instance.$el,
+        () => callUpdate(instance, 'computed'),
+        mutationOptions,
+    );
 }
 
 export function disconnectComputedUpdates(instance) {
-    instances?.delete(instance);
+    instance._computedObserver?.disconnect();
+    delete instance._computedObserver;
     resetComputed(instance);
 }
 
@@ -65,22 +78,3 @@ function resetComputed(instance) {
     instance._computed = {};
     return values;
 }
-
-let observer;
-let instances;
-function registerComputedObserver() {
-    if (observer) {
-        return;
-    }
-
-    instances = new Set();
-    observer = new MutationObserver(() => {
-        for (const instance of instances) {
-            callUpdate(instance, 'computed');
-        }
-    });
-    observer.observe(document, {
-        subtree: true,
-        childList: true,
-    });
-}

package/src/js/api/observer.js

@@ -46,21 +46,33 @@ function registerObservable(instance, observable) {
     const targets = hasOwn(instance, key) ? instance[key] : target;
     const observer = observe(targets, handler, options, args);
 
-    if (isFunction(target) && isArray(instance[key]) && observer.unobserve) {
-        registerWatch(instance, { handler: updateTargets(observer), immediate: false }, key);
+    if (isFunction(target) && isArray(instance[key])) {
+        registerWatch(
+            instance,
+            { handler: updateTargets(observer, options), immediate: false },
+            key,
+        );
     }
 
     registerObserver(instance, observer);
 }
 
-function updateTargets(observer) {
+function updateTargets(observer, options) {
     return (targets, prev) => {
         for (const target of prev) {
-            !includes(targets, target) && observer.unobserve(target);
+            if (!includes(targets, target)) {
+                if (observer.unobserve) {
+                    observer.unobserve(target);
+                } else if (observer.observe) {
+                    observer.disconnect();
+                }
+            }
         }
 
         for (const target of targets) {
-            !includes(prev, target) && observer.observe(target);
+            if (!includes(prev, target) || !observer.unobserve) {
+                observer.observe(target, options);
+            }
         }
     };
 }

package/src/js/components/slider-parallax.js

@@ -1,11 +1,11 @@
-import { closest, css, endsWith, fastdom, noop, Transition } from 'uikit-util';
+import { css, endsWith, fastdom, noop, Transition } from 'uikit-util';
 import Parallax from '../mixin/parallax';
 
 export default {
     mixins: [Parallax],
 
     beforeConnect() {
-        this.item = closest(this.$el, `.${this.$options.id.replace('parallax', 'items')} > *`);
+        this.item = this.$el.closest(`.${this.$options.id.replace('parallax', 'items')} > *`);
     },
 
     disconnected() {

package/src/js/components/slider.js

@@ -129,7 +129,7 @@ export default {
     },
 
     observe: resize({
-        target: ({ slides }) => slides,
+        target: ({ slides, $el }) => [$el, ...slides],
     }),
 
     update: {

package/src/js/core/drop.js

@@ -27,7 +27,6 @@ import {
     pointerUp,
     query,
     removeClass,
-    within,
 } from 'uikit-util';
 import Container from '../mixin/container';
 import Position from '../mixin/position';
@@ -513,7 +512,7 @@ function listenForBackgroundClose(drop) {
                     !defaultPrevented &&
                     type === pointerUp &&
                     target === newTarget &&
-                    !(drop.targetEl && within(target, drop.targetEl))
+                    !drop.targetEl?.contains(target)
                 ) {
                     drop.hide(false);
                 }

package/src/js/core/height-placeholder.js

@@ -13,7 +13,10 @@ export default {
     },
 
     computed: {
-        target: ({ target }, $el) => query(target, $el),
+        target: {
+            get: ({ target }, $el) => query(target, $el),
+            observe: ({ target }) => target,
+        },
     },
 
     observe: resize({ target: ({ target }) => target }),

package/src/js/core/height-viewport.js

@@ -68,7 +68,7 @@ export default {
                             offsetPosition(offsetTopEl)[0] - offsetPosition(scrollElement)[0];
                         minHeight += top > 0 && top < viewportHeight / 2 ? ` - ${top}px` : '';
                     } else {
-                        minHeight += ` - ${css(scrollElement, 'paddingTop')}`;
+                        minHeight += ` - ${boxModelAdjust(scrollElement, 'height', css(scrollElement, 'boxSizing'))}px`;
                     }
                 }
 

package/src/js/core/icon.js

@@ -3,7 +3,6 @@ import {
     addClass,
     apply,
     attr,
-    closest,
     css,
     each,
     hasAttr,
@@ -126,9 +125,9 @@ export const Search = {
             hasClass(this.$el, 'uk-search-toggle') || hasClass(this.$el, 'uk-navbar-toggle');
         this.icon = isToggle
             ? 'search-toggle-icon'
-            : hasClass(this.$el, 'uk-search-icon') && closest(this.$el, '.uk-search-large')
+            : hasClass(this.$el, 'uk-search-icon') && this.$el.closest('.uk-search-large')
               ? 'search-large'
-              : closest(this.$el, '.uk-search-navbar')
+              : this.$el.closest('.uk-search-navbar')
                 ? 'search-navbar'
                 : this.$props.icon;
 

package/src/js/core/img.js

@@ -56,13 +56,13 @@ export default {
     },
 
     observe: intersection({
-        target: ({ $el, $props }) => [$el, ...queryAll($props.target, $el)],
         handler(entries, observer) {
             this.load();
             observer.disconnect();
         },
         options: ({ margin }) => ({ rootMargin: margin }),
         filter: ({ loading }) => loading === 'lazy',
+        target: ({ $el, $props }) => ($props.target ? [$el, ...queryAll($props.target, $el)] : $el),
     }),
 
     methods: {

package/src/js/core/inverse.js

@@ -1,4 +1,4 @@
-import { $$, css, dimensions, matches, observeResize, on, replaceClass } from 'uikit-util';
+import { $$, css, dimensions, matches, observeResize, on, replaceClass, toNodes } from 'uikit-util';
 import { intersection, mutation } from '../api/observables';
 
 export default {
@@ -13,14 +13,16 @@ export default {
     },
 
     computed: {
-        target: ({ target }, $el) => (target ? $$(target, $el) : [$el]),
+        target: ({ target }, $el) => (target ? $$(target, $el) : $el),
     },
 
     observe: [
         intersection({
-            handler([{ isIntersecting }]) {
-                this.isIntersecting = isIntersecting;
+            handler(entries) {
+                this.isIntersecting = entries.some(({ isIntersecting }) => isIntersecting);
+                this.$emit();
             },
+            target: ({ target }) => target,
             args: { intersecting: false },
         }),
         mutation({
@@ -30,7 +32,10 @@ export default {
         {
             target: ({ target }) => target,
             observe: (target, handler) => {
-                const observer = observeResize([...target, document.documentElement], handler);
+                const observer = observeResize(
+                    [...toNodes(target), document.documentElement],
+                    handler,
+                );
                 const listener = [
                     on(document, 'scroll itemshown itemhidden', handler, {
                         passive: true,
@@ -47,6 +52,8 @@ export default {
                 ];
 
                 return {
+                    observe: observer.observe.bind(observer),
+                    unobserve: observer.unobserve.bind(observer),
                     disconnect() {
                         observer.disconnect();
                         listener.map((off) => off());
@@ -65,7 +72,7 @@ export default {
                 return false;
             }
 
-            for (const target of this.target) {
+            for (const target of toNodes(this.target)) {
                 replaceClass(
                     target,
                     'uk-light,uk-dark',

package/src/js/core/margin.js

@@ -13,6 +13,11 @@ export default {
     },
 
     observe: [
+        mutation({
+            options: {
+                childList: true,
+            },
+        }),
         mutation({
             options: {
                 attributes: true,

package/src/js/core/modal.js

@@ -70,12 +70,11 @@ export default {
 
 function install({ modal }) {
     modal.dialog = function (content, options) {
-        const dialog = modal(
-            `<div class="uk-modal">
-                <div class="uk-modal-dialog">${content}</div>
-             </div>`,
-            { stack: true, role: 'alertdialog', ...options },
-        );
+        const dialog = modal($(`<div><div class="uk-modal-dialog">${content}</div></div>`), {
+            stack: true,
+            role: 'alertdialog',
+            ...options,
+        });
 
         dialog.show();
 

package/src/js/core/scrollspy.js

@@ -135,9 +135,6 @@ export default {
             trigger(el, inview ? 'inview' : 'outview');
 
             state.inview = inview;
-
-            // change to `visibility: hidden` does not trigger observers
-            this.$update(el);
         },
     },
 };

package/src/js/core/sticky.js

@@ -109,7 +109,7 @@ export default {
             target: () => document.scrollingElement,
             options: { box: 'content-box' },
         }),
-        resize({ target: ({ $el }) => $el }),
+        resize(),
     ],
 
     events: [

package/src/js/core/switcher.js

@@ -50,7 +50,10 @@ export default {
     },
 
     computed: {
-        connects: ({ connect }, $el) => queryAll(connect, $el),
+        connects: {
+            get: ({ connect }, $el) => queryAll(connect, $el),
+            observe: ({ connect }) => connect,
+        },
 
         connectChildren() {
             return this.connects.map((el) => children(el)).flat();

package/src/js/core/toggle.js

@@ -41,9 +41,12 @@ export default {
     },
 
     computed: {
-        target({ target }, $el) {
-            target = queryAll(target || $el.hash, $el);
-            return target.length ? target : [$el];
+        target: {
+            get: ({ target }, $el) => {
+                target = queryAll(target || $el.hash, $el);
+                return target.length ? target : [$el];
+            },
+            observe: ({ target }) => target,
         },
     },
 

package/src/js/mixin/internal/animate-fade.js

@@ -11,6 +11,7 @@ import {
     Transition,
 } from 'uikit-util';
 import { getRows } from '../../core/margin';
+import { awaitFrame } from './animate-slide.js';
 
 const clsLeave = 'uk-transition-leave';
 const clsEnter = 'uk-transition-enter';
@@ -117,13 +118,7 @@ function waitTransitionend(target) {
 }
 
 function getTransitionNodes(target) {
-    return getRows(children(target))
-        .flat()
-        .filter((node) => isVisible(node));
-}
-
-function awaitFrame() {
-    return new Promise((resolve) => requestAnimationFrame(resolve));
+    return getRows(children(target)).flat().filter(isVisible);
 }
 
 function awaitTimeout(timeout) {

package/src/js/mixin/internal/animate-slide.js

@@ -2,7 +2,6 @@ import {
     attr,
     children,
     css,
-    fastdom,
     includes,
     index,
     isVisible,
@@ -26,7 +25,7 @@ export default async function (action, target, duration) {
     await Promise.all(nodes.concat(target).map(Transition.cancel));
 
     // Adding, sorting, removing nodes
-    action();
+    await action();
 
     // Find new nodes
     nodes = nodes.concat(children(target).filter((el) => !includes(nodes, el)));
@@ -34,9 +33,6 @@ export default async function (action, target, duration) {
     // Wait for update to propagate
     await Promise.resolve();
 
-    // Force update
-    fastdom.flush();
-
     // Get new state
     const targetStyle = attr(target, 'style');
     const targetPropsTo = css(target, ['height', 'padding']);
@@ -49,7 +45,6 @@ export default async function (action, target, duration) {
 
     // Trigger update in e.g. parallax component
     trigger(target, 'scroll');
-    fastdom.flush();
 
     // Start transitions on next frame
     await awaitFrame();
@@ -142,6 +137,6 @@ function getPositionWithMargin(el) {
     };
 }
 
-function awaitFrame() {
+export function awaitFrame() {
     return new Promise((resolve) => requestAnimationFrame(resolve));
 }

package/src/js/mixin/parallax.js

@@ -7,6 +7,7 @@ import {
     isString,
     isUndefined,
     noop,
+    once,
     propName,
     toFloat,
     toPx,
@@ -271,6 +272,7 @@ function setBackgroundPosFn(bgProps, positions, props) {
     };
 }
 
+const loading = {};
 const dimensions = {};
 function getBackgroundImageDimensions(el) {
     const src = css(el, 'backgroundImage').replace(/^none|url\(["']?(.+?)["']?\)$/, '$1');
@@ -283,11 +285,13 @@ function getBackgroundImageDimensions(el) {
     if (src) {
         image.src = src;
 
-        if (!image.naturalWidth) {
-            image.onload = () => {
+        if (!image.naturalWidth && !loading[src]) {
+            once(image, 'error load', () => {
                 dimensions[src] = toDimensions(image);
                 trigger(el, createEvent('load', false));
-            };
+            });
+            loading[src] = true;
+
             return toDimensions(image);
         }
     }

package/src/js/mixin/slider.js

@@ -10,7 +10,6 @@ import {
     removeClass,
     trigger,
 } from 'uikit-util';
-import { resize } from '../api/observables';
 import I18n from './i18n';
 import SliderAutoplay from './slider-autoplay';
 import SliderDrag from './slider-drag';
@@ -80,8 +79,6 @@ export default {
         },
     },
 
-    observe: resize(),
-
     events: {
         itemshow({ target }) {
             addClass(target, this.clsEnter, this.clsSlideActive);

package/src/js/mixin/slideshow.js

@@ -1,4 +1,5 @@
 import { addClass, removeClass } from 'uikit-util';
+import { resize } from '../api/observables.js';
 import Animations from './internal/slideshow-animations';
 import Transitioner from './internal/slideshow-transitioner';
 import Slider from './slider.js';
@@ -27,6 +28,8 @@ export default {
         },
     },
 
+    observe: resize(),
+
     events: {
         beforeitemshow({ target }) {
             addClass(target, this.clsActive);

package/src/js/util/class.js

@@ -55,6 +55,6 @@ function toClasses(str) {
     return str
         ? isArray(str)
             ? str.map(toClasses).flat()
-            : String(str).split(/[ ,]/).filter(Boolean)
+            : String(str).split(' ').filter(Boolean)
         : [];
 }

package/src/js/util/fastdom.js

@@ -28,28 +28,21 @@ export const fastdom = {
     flush,
 };
 
-function flush(recursion) {
+function flush() {
     runTasks(fastdom.reads);
     runTasks(fastdom.writes.splice(0));
 
     fastdom.scheduled = false;
 
     if (fastdom.reads.length || fastdom.writes.length) {
-        scheduleFlush(recursion + 1);
+        scheduleFlush();
     }
 }
 
-const RECURSION_LIMIT = 4;
-function scheduleFlush(recursion) {
-    if (fastdom.scheduled) {
-        return;
-    }
-
-    fastdom.scheduled = true;
-    if (recursion && recursion < RECURSION_LIMIT) {
-        Promise.resolve().then(() => flush(recursion));
-    } else {
-        requestAnimationFrame(() => flush(1));
+function scheduleFlush() {
+    if (!fastdom.scheduled) {
+        fastdom.scheduled = true;
+        queueMicrotask(flush);
     }
 }
 

package/src/js/util/filter.js

@@ -1,4 +1,4 @@
-import { isString, startsWith, toArray, toNode, toNodes } from './lang';
+import { toArray, toNode, toNodes } from './lang';
 
 const voidElements = {
     area: true,
@@ -21,10 +21,13 @@ export function isVoidElement(element) {
     return toNodes(element).some((element) => voidElements[element.tagName.toLowerCase()]);
 }
 
+const isVisibleFn =
+    Element.prototype.checkVisibility ||
+    function () {
+        return this.offsetWidth || this.offsetHeight || this.getClientRects().length;
+    };
 export function isVisible(element) {
-    return toNodes(element).some(
-        (element) => element.offsetWidth || element.offsetHeight || element.getClientRects().length,
-    );
+    return toNodes(element).some((element) => isVisibleFn.call(element));
 }
 
 export const selInput = 'input,select,textarea,button';
@@ -49,16 +52,6 @@ export function matches(element, selector) {
     return toNodes(element).some((element) => element.matches(selector));
 }
 
-export function closest(element, selector) {
-    return toNode(element)?.closest(startsWith(selector, '>') ? selector.slice(1) : selector);
-}
-
-export function within(element, selector) {
-    return isString(selector)
-        ? !!closest(element, selector)
-        : toNode(selector).contains(toNode(element));
-}
-
 export function parents(element, selector) {
     const elements = [];
 

package/src/js/util/lang.js

@@ -111,7 +111,7 @@ export function toFloat(value) {
 }
 
 export function toNode(element) {
-    return toNodes(element)[0];
+    return element && toNodes(element)[0];
 }
 
 export function toNodes(element) {