ng-drag-drop icon indicating copy to clipboard operation
ng-drag-drop copied to clipboard

Sortable: Multi list sorting bug

Open ObaidUrRehman opened this issue 8 years ago • 10 comments
trafficstars

Item does not get removed from the list when sorted between multiple lists. See below:

multilist-sorable-bug

ObaidUrRehman avatar Aug 08 '17 12:08 ObaidUrRehman

@ObaidUrRehman I'm unable to reproduce this in chrome, firefox or safari (all latest versions) on my mac. Can you please specify which browser and OS you're experiencing this on?

cah-alexsukhodolsky avatar Aug 08 '17 15:08 cah-alexsukhodolsky

I am using Chrome Version 60.0.3112.90 on Window. I'll see if I can come up with steps to reproduce this.

ObaidUrRehman avatar Aug 09 '17 07:08 ObaidUrRehman

I started looking into this and I'm able to reproduce it in Windows. This issue is most likely caused by a race condition. What would you think about queueing up any dom changes and performing them one at a time?

cah-alexsukhodolsky avatar Aug 16 '17 14:08 cah-alexsukhodolsky

If this is a race condition, then it comes up too often. I am trying to come up with steps to reproduce. Can you elaborate a bit on how we can queue up DOM changes?

ObaidUrRehman avatar Aug 17 '17 11:08 ObaidUrRehman

sorry...been off the radar the last few days. I'll look into the problem and solution this week.

cah-alexsukhodolsky avatar Aug 22 '17 18:08 cah-alexsukhodolsky

@ObaidUrRehman Were you able to reproduce this bug? Did you find my changes fixed the issue?

cah-alexsukhodolsky avatar Sep 06 '17 16:09 cah-alexsukhodolsky

@cah-alexsukhodolsky I am pretty slammed nowadays. Give me a few days and I will get back to you on this.

ObaidUrRehman avatar Sep 06 '17 17:09 ObaidUrRehman

@ObaidUrRehman any update on this?

cah-alexsukhodolsky avatar Sep 25 '17 15:09 cah-alexsukhodolsky

@cah-alexsukhodolsky sorry for the delay on this.

I've tested this and unfortunately, it still occurs. I don't have the steps to reproduce this but dragging items back and forth repeatedly causes this issue.

ObaidUrRehman avatar Oct 11 '17 09:10 ObaidUrRehman

please refer to: https://github.com/vinayakpatil/ng-drag-drop/tree/feat/sortable

vinayakpatil avatar Jul 03 '18 19:07 vinayakpatil