Skip to content

Commit

Permalink
Draggable: Calling refreshPositions instead of _refreshItems when dra…
Browse files Browse the repository at this point in the history
…gging into a connected sortable. Fixed: #5563 - Draggable li with connectToSortable option cannot be dropped on the connected sortable ol
  • Loading branch information
nathansobo committed Jul 8, 2010
1 parent 89c0961 commit 3eadf2b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ui/jquery.ui.draggable.js
Expand Up @@ -475,7 +475,7 @@ $.ui.plugin.add("draggable", "connectToSortable", {
instance: sortable,
shouldRevert: sortable.options.revert
});
sortable._refreshItems(); //Do a one-time refresh at start to refresh the containerCache
sortable.refreshPositions(); //Do a one-time refreshPositions at start to initialize the containerCache
sortable._trigger("activate", event, uiSortable);
}
});
Expand Down

1 comment on commit 3eadf2b

@FurLined
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The author of the patch noted it fixed 5563 and I've tested it against the test case attached there - the patch also fixes what's reported in 5811 as that's the same bug.

Please sign in to comment.