From: Scott González Date: Fri, 12 Aug 2011 14:06:52 +0000 (-0400) Subject: Droppable: Don't use .parentsUntil(). Fixes #7530 - Droppable fails on a.element... X-Git-Tag: 1.8.16~1 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=7b3e6a85c189e703573edd662ced72a3d5eec425;p=jquery-ui.git Droppable: Don't use .parentsUntil(). Fixes #7530 - Droppable fails on a.element.parentsuntil in version 1.8.14. --- diff --git a/ui/jquery.ui.droppable.js b/ui/jquery.ui.droppable.js index 58a1c4bc1..dabf29d6c 100644 --- a/ui/jquery.ui.droppable.js +++ b/ui/jquery.ui.droppable.js @@ -240,7 +240,7 @@ $.ui.ddmanager = { }, dragStart: function( draggable, event ) { //Listen for scrolling so that if the dragging causes scrolling the position of the droppables can be recalculated (see #5003) - draggable.element.parentsUntil( "body" ).bind( "scroll.droppable", function() { + draggable.element.parents( ":not(body,html)" ).bind( "scroll.droppable", function() { if( !draggable.options.refreshPositions ) $.ui.ddmanager.prepareOffsets( draggable, event ); }); }, @@ -287,7 +287,7 @@ $.ui.ddmanager = { }, dragStop: function( draggable, event ) { - draggable.element.parentsUntil( "body" ).unbind( "scroll.droppable" ); + draggable.element.parents( ":not(body,html)" ).unbind( "scroll.droppable" ); //Call prepareOffsets one final time since IE does not fire return scroll events when overflow was caused by drag (see #5003) if( !draggable.options.refreshPositions ) $.ui.ddmanager.prepareOffsets( draggable, event ); }