vue.draggable.next
vue.draggable.next copied to clipboard
Fix for issue 34, switched this.$el with evt.from
onDragUpdate was updated to use evt.from to call insertNodeAt, but onDragRemove was not updated accordingly.
This pull request simply corrects this slight oversight, fixing a crash I was experiencing.
@alnumac Could you specify which scenario this fix is correcting?
@David-Desmaisons Moving an item from one list to another with animations enabled.
Yes, i have the same problem, and looking for this. fix. Have to disable animation for current version. There is no such problem for previous vue2 version.
@alnumac I will appreciate if you can do this fix. i use it here: https://github.com/cybersturmer/pmdragon-client/blob/cee26d62eb86b031c4ddb8d567e10cedf154271b/src/pages/dash/Backlog.vue#L193
and here: https://github.com/cybersturmer/pmdragon-client/blob/cee26d62eb86b031c4ddb8d567e10cedf154271b/src/pages/dash/Board.vue#L131
Please correct this bug and push a new release! Thank you So much!
I also would be in favor of merging and releasing this as I just now ran into the same issue while upgrading a vue2 project to vue3 and moving items between lists of the same group throws the Cannot read properties of undefined error inside the onDragRemove function, I'm also using the transition-group with the animation prop.
only workaround for now is removing the transition-group and the animation prop
@David-Desmaisons what is missing for this to be merged?
Please merge this PR and push a new release! Thanks very much!