Draggable: Ensure parent is correct after dragging through sortable

Fixes #10669

diff --git a/tests/unit/draggable/draggable_options.js b/tests/unit/draggable/draggable_options.js
index e54ae1df46d8d31732f3837b4c679c227a2dbc41..d8bbedc9163a2b6cdb7c2b92dfca9e13655cc581 100644 (file)
--- a/tests/unit/draggable/draggable_options.js
+++ b/tests/unit/draggable/draggable_options.js
@@ -405,6 +405,29 @@ test( "connectToSortable, dragging through one sortable to a second", function()
        element.simulate( "drag", dragParams );
 });
 
+test( "connectToSortable, dragging through a sortable", function() {
+       expect( 1 );
+
+       var draggable = $( "#draggableSortable" ).draggable({
+                       scroll: false,
+                       connectToSortable: "#sortable2"
+               }),
+               sortable = $( "#sortable2" ).sortable(),
+               sortableOffset = sortable.offset();
+
+       // http://bugs.jqueryui.com/ticket/10669
+       // Draggable: Position issue with connectToSortable
+       draggable.one( "dragstop", function() {
+               equal( draggable.parent().attr( "id" ), "sortable", "restored draggable to original parent" );
+       });
+
+       draggable.simulate( "drag", {
+               x: sortableOffset.left + 25,
+               y: sortableOffset.top + sortable.outerHeight() + 400,
+               moves: 20
+       });
+});
+
 test( "{ containment: Element }", function() {
        expect( 1 );
 

diff --git a/ui/draggable.js b/ui/draggable.js
index b59955add554f0e2bab08f98ab4eb35548170966..5be628dc8c53c5c4602eff4664e66bd7ebf858f9 100644 (file)
--- a/ui/draggable.js
+++ b/ui/draggable.js
@@ -800,6 +800,9 @@ $.ui.plugin.add( "draggable", "connectToSortable", {
                                if ( !sortable.isOver ) {
                                        sortable.isOver = 1;
 
+                                       // Store draggable's parent in case we need to reappend to it later.
+                                       draggable._parent = ui.helper.parent();
+
                                        sortable.currentItem = ui.helper
                                                .appendTo( sortable.element )
                                                .data( "ui-sortable-item", true );
@@ -876,8 +879,9 @@ $.ui.plugin.add( "draggable", "connectToSortable", {
                                                sortable.placeholder.remove();
                                        }
 
-                                       // Recalculate the draggable's offset considering the sortable
-                                       // may have modified them in unexpected ways (#8809)
+                                       // Restore and recalculate the draggable's offset considering the sortable
+                                       // may have modified them in unexpected ways. (#8809, #10669)
+                                       ui.helper.appendTo( draggable._parent );
                                        draggable._refreshOffsets( event );
                                        ui.position = draggable._generatePosition( event, true );