Skip to main content

Search and Top Navigation

#9675 closed bug (fixed)

Opened November 25, 2013 07:30PM UTC

Closed August 26, 2014 11:25PM UTC

Last modified August 26, 2014 11:26PM UTC

Draggable: Animation issue with revert and connectToSortable

Reported by: tapapax Owned by: mikesherov
Priority: minor Milestone: 1.11.2
Component: ui.draggable Version: 1.10.3
Keywords: Cc:
Blocked by: Blocking:
Description

Hi!

I've got some issue with draggable, which is connected with multiple sortables. The issue appears in two ways:

1) If I'm dragging an item from draggable _throw_ some sortable, "revert" option of this sortable become false (at jquery-ui.js:1685).

2) If mouse cursor skipped from one sortable to another while dragging (without leaving sortables area), then item will move a bit weird after releasing mouse button. It will first return to it's original place (at it's draggable), and after that it will move to the sortable (this is right).

Versions 1.10.3 and 1.8.0. All major browsers.

I am very sorry for my bad english.

link: http://jsbin.com/odiqi3

video 1: http://list.dominosoft.ru/media/revert.wmv

video 2: http://list.dominosoft.ru/media/weirdMovement.wmv

Attachments (0)
Change History (11)

Changed November 25, 2013 08:18PM UTC by tj.vantoll comment:1

owner: → tapapax
status: newpending

Hi tapapax,

Thanks for taking the time to contribute to the jQuery UI project. Your first concern is covered by bug #9481. For your second issue I believe it has already been reported in #9099. Could you confirm that's the same issue you're seeing?

Thanks.

Changed November 25, 2013 08:48PM UTC by tapapax comment:2

status: pendingnew

Hi tj.vantoll,

Thank you.

Right, this http://bugs.jqueryui.com/ticket/9099#comment:8 comment seems similar, but I think it is not the same issue. Among others, it is not describing animation problem, which can be seen on my video http://list.dominosoft.ru/media/weirdMovement.wmv

Changed November 25, 2013 09:49PM UTC by tj.vantoll comment:3

status: newpending

Ok thanks.

Are you seeing the problem in the JS Bin test case you provided? I can't recreate it there. Also, can you update it to use the following files and see if the problem still occurs (it's the latest code from the master branch from the git repo).

<link href="http://code.jquery.com/ui/jquery-ui-git.css" rel="stylesheet">
<script src="http://code.jquery.com/jquery-git.js"></script>
<script src="http://code.jquery.com/ui/jquery-ui-git.js"></script>

Thanks.

Changed November 25, 2013 10:18PM UTC by tapapax comment:4

status: pendingnew

Yes, there is a problem in it. I changed this test case a bit, now http://jsbin.com/odiqi3/176/ it is easier to reproduce the bug. Just drag draggable inside top sortable first, and than (you should do it fast) move it inside bottom one. Maybe you will need a few attempts to get this behaviour.

Demo: http://list.dominosoft.ru/media/demo.avi

New test case is using jquery-ui from git repo, but the problem is still the same.

Thanks!

Changed November 26, 2013 02:08PM UTC by tj.vantoll comment:5

component: ui.coreui.draggable
status: newopen
summary: Draggable with multiple sortablesDraggable: Animation issue with revert and connectToSortable

Thanks for sticking with this tapapax. I see the issue now.

Changed August 24, 2014 12:59PM UTC by mikesherov comment:6

owner: tapapaxmikesherov
status: openassigned

Well, this one is annoying but I'm working on this fix now.

Changed August 26, 2014 11:25PM UTC by Mike Sherov comment:7

resolution: → fixed
status: assignedclosed

Draggable: Ensure css is always restored after connectToSortable drag

Fixes #9675

Changeset: 27ed20715f4b3e256f5279825ac551bbfcdfbe81

Changed August 26, 2014 11:25PM UTC by Mike Sherov comment:8

Draggable: Refresh sortables when draggable is added or removed

Since a sortable grows or shrinks when a draggable element is added

to it, refresh the cached positions of sortables whenever an element

is added or removed from the sortable.

Refs #9675

Changeset: a611dd8971a5fada1ca9e661ad1944b401192f0d

Changed August 26, 2014 11:25PM UTC by Mike Sherov comment:9

Sortable: cancelHelperRemoval only considers helper, not placeholder

Refs #9675

Changeset: 52a1de5caadd9dd0665d4bf092f6061d9d3a3a8e

Changed August 26, 2014 11:25PM UTC by Mike Sherov comment:10

Draggable: Add battery of tests to cover connectToSortable

Refs #9481

Refs #9675

Closes gh-1323

Changeset: bfb65076e4727d4d7fc88561b0304853accb925c

Changed August 26, 2014 11:26PM UTC by mikesherov comment:11

milestone: none1.11.2