Skip to content
Permalink
Browse files

Draggable: Handle scrolled windows. Fixes #6181 - Draggable: issue wi…

…th containment on position:fixed elements.
  • Loading branch information
Jean-Francois Remy authored and scottgonzalez committed Dec 1, 2010
1 parent eac910f commit e01664a3820c185636a3fe7099e93e68091d24fd
Showing with 4 additions and 4 deletions.
  1. +4 −4 ui/jquery.ui.draggable.js
@@ -333,10 +333,10 @@ $.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 = [
0 - this.offset.relative.left - this.offset.parent.left,
0 - this.offset.relative.top - this.offset.parent.top,
$(o.containment == 'document' ? document : window).width() - this.helperProportions.width - this.margins.left,
($(o.containment == 'document' ? document : window).height() || document.body.parentNode.scrollHeight) - this.helperProportions.height - this.margins.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
];

if(!(/^(document|window|parent)$/).test(o.containment) && o.containment.constructor != Array) {

1 comment on commit e01664a

@yasinuslu

This comment has been minimized.

Copy link

@yasinuslu yasinuslu commented on e01664a Oct 15, 2012

i'm still having http://bugs.jqueryui.com/ticket/6181 issue

Please sign in to comment.
You can’t perform that action at this time.