From 24f331568374c20115003d089664a02b8df74590 Mon Sep 17 00:00:00 2001 From: kborchers Date: Mon, 16 May 2011 21:08:16 -0500 Subject: [PATCH] Droppable: Make the undocumented refreshPositions option standard to recalculate positions. Fixed #5003 - Scroll on Droppable Demo Breaks Demo --- ui/jquery.ui.droppable.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ui/jquery.ui.droppable.js b/ui/jquery.ui.droppable.js index b8a93cd460b..0ea922b595e 100644 --- a/ui/jquery.ui.droppable.js +++ b/ui/jquery.ui.droppable.js @@ -240,8 +240,9 @@ $.ui.ddmanager = { }, drag: function(draggable, event) { - //If you have a highly dynamic page, you might try this option. It renders positions every time you move the mouse. - if(draggable.options.refreshPositions) $.ui.ddmanager.prepareOffsets(draggable, event); + //Renders positions every time you move the mouse. Necessary when dragging causes scroll which gives the + //droppable negative positions that must be recalculated. (see ticket #5003) + $.ui.ddmanager.prepareOffsets(draggable, event); //Run through all droppables and check their positions based on specific tolerance options $.each($.ui.ddmanager.droppables[draggable.options.scope] || [], function() {