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
Attachments (0)
Change History (11)
Changed November 25, 2013 08:18PM UTC by comment:1
| owner: | → tapapax |
|---|---|
| status: | new → pending |
Changed November 25, 2013 08:48PM UTC by comment:2
| status: | pending → new |
|---|
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 comment:3
| status: | new → pending |
|---|
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 comment:4
| status: | pending → new |
|---|
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 comment:5
| component: | ui.core → ui.draggable |
|---|---|
| status: | new → open |
| summary: | Draggable with multiple sortables → Draggable: 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 comment:6
| owner: | tapapax → mikesherov |
|---|---|
| status: | open → assigned |
Well, this one is annoying but I'm working on this fix now.
Changed August 26, 2014 11:25PM UTC by comment:7
| resolution: | → fixed |
|---|---|
| status: | assigned → closed |
Draggable: Ensure css is always restored after connectToSortable drag
Fixes #9675
Changeset: 27ed20715f4b3e256f5279825ac551bbfcdfbe81
Changed August 26, 2014 11:25PM UTC by 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 comment:9
Sortable: cancelHelperRemoval only considers helper, not placeholder
Refs #9675
Changeset: 52a1de5caadd9dd0665d4bf092f6061d9d3a3a8e
Changed August 26, 2014 11:25PM UTC by comment:10
Changed August 26, 2014 11:26PM UTC by comment:11
| milestone: | none → 1.11.2 |
|---|
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.