Fix Copy Option

Hey @mfriesen there is an issue with the copy option: https://lukasoppermann.github.io/html5sortable/index.html

• The new items that are added to a sortable are not part of it, so we would need to reinit every time a copy is added to an item. (This is not so good, maybe we could fix this by fixing the general issue: adding new items to a sortable should automatically work, without the reinit: #312)
• A bug in copy is that when I move a copied item onto an invalid target (and my copy start list has acceptFrom:false) the item still goes back to the parent. This could also be seen in a bigger picture so that dropping outside a valid target should cancel the sorting action (similar to when pressing esc #313)

[mfriesen]

@lukasoppermann fixing this bug is what I’m trying to do. But in order to do it properly I need to extract break up the code inside of the

sortableElements.forEach(function (sortableElement) {

After my pull requests are merged, I will extract the

_on(items, 'drag event .,.', function (e)

to their own function. Then I believe I can fix this problem.

[mfriesen]

@lukasoppermann I’ll take a look at the reinit problem. I should have caught that problem.

For the second part are you saying that once you drop the ‘copy’ on a valid target the item is pointing to the original parent?

The other thing I just noticed was that the data-item-sortable-id doesn’t change when moving items between 2 sortables. Is that ok?