#1954: Add useAnimation option to sort()

SortableJS · Jan 8, 2021 · 5afb802 · 5afb802
1 parent a0e160a
commit 5afb802
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -565,12 +565,12 @@ For each element in the set, get the first element that matches the selector by
 Serializes the sortable's item `data-id`'s (`dataIdAttr` option) into an array of string.
 
 
-##### sort(order:`String[]`)
+##### sort(order:`String[]`, useAnimation:`Boolean`)
 Sorts the elements according to the array.
 
 ```js
 var order = sortable.toArray();
-sortable.sort(order.reverse()); // apply
+sortable.sort(order.reverse(), true); // apply
 ```
 
 

diff --git a/src/Sortable.js b/src/Sortable.js
@@ -1569,7 +1569,7 @@ Sortable.prototype = /** @lends Sortable.prototype */ {
 	 * Sorts the elements according to the array.
 	 * @param  {String[]}  order  order of the items
 	 */
-	sort: function (order) {
+	sort: function (order, useAnimation) {
 		let items = {}, rootEl = this.el;
 
 		this.toArray().forEach(function (id, i) {
@@ -1580,12 +1580,14 @@ Sortable.prototype = /** @lends Sortable.prototype */ {
 			}
 		}, this);
 
+		useAnimation && this.captureAnimationState();
 		order.forEach(function (id) {
 			if (items[id]) {
 				rootEl.removeChild(items[id]);
 				rootEl.appendChild(items[id]);
 			}
 		});
+		useAnimation && this.animateAll();
 	},