bug introduced in 2.16.0 while dragging with revert position [proposed solution inside]

[chino23]

#567 introduced a fix for the revert position. However, the way my table is set up, it produces a major error which seems easy to fix.

The added code of #567 reads:

if( dnd.smartRevert ) {
    // #567: fix revert position
    rect = node.li.getBoundingClientRect();
    parentRect = $(draggable.options.appendTo)[0].getBoundingClientRect();
    draggable.originalPosition.left = Math.max(0, rect.left - parentRect.left);
    draggable.originalPosition.top = Math.max(0, rect.top - parentRect.top);
}

however, I'm running the DND extension with tables. Therefore,

node.li -> null
node.tr -> I'm here

changing

node.li

to

node.tr

fixes the problem for me.
So I guess we need a check if node.li is null, that we use node.tr.

[allenfantasy]

Found the same issue with the OP, and I posted the detail in #114

[mar10]

Thanks for reporting!

[allenfantasy]

👍

[chino23]

Thanks for the quick fix!