Items are missing on drop

[madanv]

Hello, I observed that the items are sometimes missing after drop into a different list from its parent. Could some one help me resolving it.

[madanv]

Attached a sample screenshot:

[madanv]

Hi, I have figured out that the problem for me is actually with the removeclass function in dragend event.

element.removeClass("dndDraggingSource");

The classname, dndDraggingSource is added to the element in dragstart event using by timeout. In my case when I try to drag an element and drop it on the screen in a quick flash, dragend event is working fine but the dndDraggingSource class is being added after dragend event because of timeout.

I was able to fix this by updating the dragend event to remove the class, dndDraggingSource by using timeout as below.

$timeout(function () { element.removeClass("dndDraggingSource"); }, 0);

If this makes sense could this be fixed in the module as well.

[yohanyflores]

I really the same thing happened, especially in chrome. This happens because sometimes the "dragend" event is sent without triggers the "drop" event, and also set event.dataTransfer.dropEffect to a value other than 'none'.

This causes the element to be lost, because it is removed without first copying.

[madanv]

@yohanyflores any suggestion for fixing it?

[yohanyflores]

I always use the fallback, dndDropEffectWorkaround.dropEffect.

In "dragend"

dropEffect = dndDropEffectWorkaround.dropEffect; 

In "drop"

          // In Chrome on Windows the dropEffect will always be none...
          // We have to determine the actual effect manually from the allowed effects
          if (event.dataTransfer.dropEffect === "none") {
            dndDropEffectWorkaround.dropEffect = event.dataTransfer.effectAllowed === "copyMove"
              ? (event.ctrlKey ? "copy" : "move")
              :  event.dataTransfer.effectAllowed;
          } else {
            dndDropEffectWorkaround.dropEffect = event.dataTransfer.dropEffect;
          }

[madanv]

@yohanyflores thanks for the help.

[marceljuenemann]

Thanks a lot for the great work! I've also seen the problem sometimes, but I wasn't able to reproduce it reliably. When I find a few hours, I'll review this and release a new version (takes a while since I want to test each release on all major operating systems and browsers)