DnD Snap to Animation fix

fix for #1973
* added a timeout > 0 to reset animation back so values have a chance to render...

Author: adumesny

doc/CHANGES.md

@@ -5,6 +5,7 @@ Change log
 <!-- DON'T EDIT THIS SECTION, INSTEAD RE-RUN doctoc TO UPDATE -->
 **Table of Contents**  *generated with [DocToc](http://doctoc.herokuapp.com/)*
 
+- [5.1.0-dev (TBD)](#510-dev-tbd)
 - [5.1.0 (2022-05-21)](#510-2022-05-21)
 - [5.0.0 (2022-01-10)](#500-2022-01-10)
 - [4.4.1 (2021-12-24)](#441-2021-12-24)
@@ -67,6 +68,8 @@ Change log
 
 <!-- END doctoc generated TOC please keep comment here to allow auto update -->
 
+## 5.1.0-dev (TBD)
+* fix v5.1.0 regression [#1973](https://github.com/gridstack/gridstack.js/issues/1973) DnD Snap to Animation
 
 ## 5.1.0 (2022-05-21)
 * add `GridStack.registerEngine()` to let user use their own custom layout engine subclass. Thank you [Thomas] for sponsoring it.

src/h5/dd-draggable.ts

@@ -236,11 +236,11 @@ export class DDDraggable extends DDBaseImplement implements HTMLElementExtendOpt
     // don't bother restoring styles if we're gonna remove anyway...
     if (this.dragElementOriginStyle && (!node || !node._isAboutToRemove)) {
       let helper = this.helper;
-      DDDraggable.originStyleProp.forEach(prop => helper.style[prop] = this.dragElementOriginStyle[prop] || null);
-      // show up instantly otherwise we animate to off the grid when switching back to 'absolute' from 'fixed'
-      helper.style.transition = 'none';
+      // don't animate, otherwise we animate offseted when switching back to 'absolute' from 'fixed' 
       let transition = this.dragElementOriginStyle['transition'] || null;
-      setTimeout(() => helper.style.transition = transition, 0); // recover animation from saved vars
+      helper.style.transition = this.dragElementOriginStyle['transition'] = 'none';
+      DDDraggable.originStyleProp.forEach(prop => helper.style[prop] = this.dragElementOriginStyle[prop] || null);
+      setTimeout(() => helper.style.transition = transition, 50); // recover animation from saved vars after a pause (0 isn't enough #1973)
     }
     delete this.dragElementOriginStyle;
     return this;