uikit 3.20.8-dev.a340cc6f4 → 3.20.8-dev.c77f59741

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.20.8-dev.a340cc6f4",
+  "version": "3.20.8-dev.c77f59741",
   "main": "dist/js/uikit.js",
   "style": "dist/css/uikit.css",
   "sideEffects": [

package/src/js/components/sortable.js

@@ -65,6 +65,16 @@ export default {
         pos: {},
     },
 
+    created() {
+        for (const key of ['init', 'start', 'move', 'end']) {
+            const fn = this[key];
+            this[key] = (e) => {
+                assign(this.pos, getEventPos(e));
+                fn(e);
+            };
+        }
+    },
+
     events: {
         name: pointerDown,
         passive: false,
@@ -184,7 +194,6 @@ export default {
 
             e.preventDefault();
 
-            this.pos = getEventPos(e);
             this.touched = new Set([this]);
             this.placeholder = placeholder;
             this.origin = { target, index: index(placeholder), ...this.pos };
@@ -214,9 +223,7 @@ export default {
             this.move(e);
         },
 
-        move: throttle(function (e) {
-            assign(this.pos, getEventPos(e));
-
+        move(e) {
             if (this.drag) {
                 this.$emit('move');
             } else if (
@@ -225,7 +232,7 @@ export default {
             ) {
                 this.start(e);
             }
-        }),
+        },
 
         end() {
             off(document, pointerMove, this.move);
@@ -265,17 +272,17 @@ export default {
         insert(element, target) {
             addClass(this.items, this.clsItem);
 
-            if (target && target.previousElementSibling !== element) {
-                this.animate(() => before(target, element));
-            } else if (!target && this.target.lastElementChild !== element) {
-                this.animate(() => append(this.target, element));
-            }
+            const insert = () => (target ? before(target, element) : append(this.target, element));
+
+            this.animate(insert);
         },
 
         remove(element) {
-            if (this.target.contains(element)) {
-                this.animate(() => remove(element));
+            if (!this.target.contains(element)) {
+                return;
             }
+
+            this.animate(() => remove(element));
         },
 
         getSortable(element) {
@@ -431,16 +438,3 @@ function isHorizontal(list, placeholder) {
 function linesIntersect(lineA, lineB) {
     return lineA[1] > lineB[0] && lineB[1] > lineA[0];
 }
-
-function throttle(fn) {
-    let throttled;
-    return function (...args) {
-        if (!throttled) {
-            throttled = true;
-            requestAnimationFrame(() => {
-                throttled = false;
-                fn.call(this, ...args);
-            });
-        }
-    };
-}

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

@@ -21,13 +21,8 @@ export default async function (action, target, duration) {
     const currentProps = nodes.map((el) => getProps(el, true));
     const targetProps = { ...css(target, ['height', 'padding']), display: 'block' };
 
-    const targets = nodes.concat(target);
-
     // Cancel previous animations
-    await Promise.all(targets.map(Transition.cancel));
-
-    // Force transition to be canceled in Safari
-    css(targets, 'transitionProperty', 'none');
+    await Promise.all(nodes.concat(target).map(Transition.cancel));
 
     // Adding, sorting, removing nodes
     await action();
@@ -38,9 +33,6 @@ export default async function (action, target, duration) {
     // Wait for update to propagate
     await Promise.resolve();
 
-    // Possibly reset the forced transition property
-    css(targets, 'transitionProperty', '');
-
     // Get new state
     const targetStyle = attr(target, 'style');
     const targetPropsTo = css(target, ['height', 'padding']);