Skip to content

Commit d000256

Browse files
committed
Droppable: Moved isOverAxis() into $.ui.intersect().
1 parent d6f8564 commit d000256

File tree

1 file changed

+22
-21
lines changed

1 file changed

+22
-21
lines changed

ui/jquery.ui.droppable.js

Lines changed: 22 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,6 @@
1616
*/
1717
(function( $, undefined ) {
1818

19-
function isOverAxis( x, reference, size ) {
20-
return ( x >= reference ) && ( x < ( reference + size ) );
21-
}
22-
2319
$.widget( "ui.droppable", {
2420
version: "@VERSION",
2521
widgetEventPrefix: "drop",
@@ -204,23 +200,28 @@ $.widget( "ui.droppable", {
204200

205201
});
206202

207-
$.ui.intersect = function( draggable, droppable, toleranceMode ) {
208-
209-
if ( !droppable.offset ) {
210-
return false;
203+
$.ui.intersect = (function() {
204+
function isOverAxis( x, reference, size ) {
205+
return ( x >= reference ) && ( x < ( reference + size ) );
211206
}
212207

213-
var draggableLeft, draggableTop,
214-
x1 = ( draggable.positionAbs || draggable.position.absolute ).left,
215-
y1 = ( draggable.positionAbs || draggable.position.absolute ).top,
216-
x2 = x1 + draggable.helperProportions.width,
217-
y2 = y1 + draggable.helperProportions.height,
218-
l = droppable.offset.left,
219-
t = droppable.offset.top,
220-
r = l + droppable.proportions().width,
221-
b = t + droppable.proportions().height;
222-
223-
switch ( toleranceMode ) {
208+
return function( draggable, droppable, toleranceMode ) {
209+
210+
if ( !droppable.offset ) {
211+
return false;
212+
}
213+
214+
var draggableLeft, draggableTop,
215+
x1 = ( draggable.positionAbs || draggable.position.absolute ).left,
216+
y1 = ( draggable.positionAbs || draggable.position.absolute ).top,
217+
x2 = x1 + draggable.helperProportions.width,
218+
y2 = y1 + draggable.helperProportions.height,
219+
l = droppable.offset.left,
220+
t = droppable.offset.top,
221+
r = l + droppable.proportions().width,
222+
b = t + droppable.proportions().height;
223+
224+
switch ( toleranceMode ) {
224225
case "fit":
225226
return ( l <= x1 && x2 <= r && t <= y1 && y2 <= b );
226227
case "intersect":
@@ -245,8 +246,8 @@ $.ui.intersect = function( draggable, droppable, toleranceMode ) {
245246
default:
246247
return false;
247248
}
248-
249-
};
249+
};
250+
})();
250251

251252
/*
252253
This manager tracks offsets of draggables and droppables

0 commit comments

Comments
 (0)