Permalink
Browse files

Droppable: Don't use .parentsUntil(). Fixes #7530 - Droppable fails o…

…n a.element.parentsuntil in version 1.8.14.
  • Loading branch information...
1 parent 930c650 commit 7b3e6a85c189e703573edd662ced72a3d5eec425 @scottgonzalez scottgonzalez committed Aug 12, 2011
Showing with 2 additions and 2 deletions.
  1. +2 −2 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 );
}

0 comments on commit 7b3e6a8

Please sign in to comment.