Skip to content

Commit 98a98a0

Browse files
committed
Dialog: Don't call draggable.destroy if we're not currently draggable. Fixes #6069 - DIALOG windows 1.8.5 draggapble problem.
1 parent 82b1595 commit 98a98a0

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

ui/jquery.ui.dialog.js

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -521,7 +521,8 @@ $.widget("ui.dialog", {
521521
_setOption: function(key, value){
522522
var self = this,
523523
uiDialog = self.uiDialog,
524-
isResizable = uiDialog.is(':data(resizable)'),
524+
isDraggable = uiDialog.is( ":data(draggable)" ),
525+
isResizable = uiDialog.is( ":data(resizable)" ),
525526
resize = false;
526527

527528
switch (key) {
@@ -552,10 +553,12 @@ $.widget("ui.dialog", {
552553
}
553554
break;
554555
case "draggable":
555-
if (value) {
556+
if ( isDraggable && !value ) {
557+
uiDialog.draggable( "destroy" );
558+
}
559+
560+
if ( !isDraggable && value ) {
556561
self._makeDraggable();
557-
} else {
558-
uiDialog.draggable('destroy');
559562
}
560563
break;
561564
case "height":

0 commit comments

Comments
 (0)