diff options
author | kborchers <k_borchers@yahoo.com> | 2011-05-12 16:27:12 -0500 |
---|---|---|
committer | kborchers <k_borchers@yahoo.com> | 2011-05-12 16:27:12 -0500 |
commit | 6e6d0b749c4724955a88a0a53ae1378391991064 (patch) | |
tree | 3896c7bebf40499d7008b8f3a864ef9497b4207f /ui | |
parent | 1845015c09d527abde8095f8e2209f06d879a2b3 (diff) | |
download | jquery-ui-6e6d0b749c4724955a88a0a53ae1378391991064.tar.gz jquery-ui-6e6d0b749c4724955a88a0a53ae1378391991064.zip |
Draggable: Changed logic to only subtract left and top offsets of containment if left and top are not already 0. Fixed #4838 - containment: 'document' is incorrect for top and left in draggable
Diffstat (limited to 'ui')
-rw-r--r-- | ui/jquery.ui.draggable.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ui/jquery.ui.draggable.js b/ui/jquery.ui.draggable.js index d7bc09578..acc6c7bb3 100644 --- a/ui/jquery.ui.draggable.js +++ b/ui/jquery.ui.draggable.js @@ -355,8 +355,8 @@ $.widget("ui.draggable", $.ui.mouse, { var o = this.options; if(o.containment == 'parent') o.containment = this.helper[0].parentNode; if(o.containment == 'document' || o.containment == 'window') this.containment = [ - (o.containment == 'document' ? 0 : $(window).scrollLeft()) - this.offset.relative.left - this.offset.parent.left, - (o.containment == 'document' ? 0 : $(window).scrollTop()) - this.offset.relative.top - this.offset.parent.top, + o.containment == 'document' ? 0 : $(window).scrollLeft() - this.offset.relative.left - this.offset.parent.left, + o.containment == 'document' ? 0 : $(window).scrollTop() - this.offset.relative.top - this.offset.parent.top, (o.containment == 'document' ? 0 : $(window).scrollLeft()) + $(o.containment == 'document' ? document : window).width() - this.helperProportions.width - this.margins.left, (o.containment == 'document' ? 0 : $(window).scrollTop()) + ($(o.containment == 'document' ? document : window).height() || document.body.parentNode.scrollHeight) - this.helperProportions.height - this.margins.top ]; |