Skip to content

Commit 3350e6c

Browse files
committed
Draggable: Add classes option
1 parent ed52ec2 commit 3350e6c

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

tests/unit/draggable/draggable_common.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ TestHelpers.commonWidgetTests( "draggable", {
33
appendTo: "parent",
44
axis: false,
55
cancel: "input,textarea,button,select,option",
6+
classes: {},
67
connectToSortable: false,
78
containment: false,
89
cursor: "auto",

ui/draggable.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ $.widget("ui.draggable", $.ui.mouse, {
3232
addClasses: true,
3333
appendTo: "parent",
3434
axis: false,
35+
classes: {},
3536
connectToSortable: false,
3637
containment: false,
3738
cursor: "auto",
@@ -65,10 +66,10 @@ $.widget("ui.draggable", $.ui.mouse, {
6566
this._setPositionRelative();
6667
}
6768
if (this.options.addClasses){
68-
this.element.addClass("ui-draggable");
69+
this._addClass( "ui-draggable" );
6970
}
7071
if (this.options.disabled){
71-
this.element.addClass("ui-draggable-disabled");
72+
this._addClass( "ui-draggable-disabled" );
7273
}
7374
this._setHandleClassName();
7475

@@ -88,7 +89,6 @@ $.widget("ui.draggable", $.ui.mouse, {
8889
this.destroyOnClear = true;
8990
return;
9091
}
91-
this.element.removeClass( "ui-draggable ui-draggable-dragging ui-draggable-disabled" );
9292
this._removeHandleClassName();
9393
this._mouseDestroy();
9494
},
@@ -164,7 +164,7 @@ $.widget("ui.draggable", $.ui.mouse, {
164164
//Create and append the visible helper
165165
this.helper = this._createHelper(event);
166166

167-
this.helper.addClass("ui-draggable-dragging");
167+
this._addClass( this.helper, "ui-draggable-dragging" );
168168

169169
//Cache the helper size
170170
this._cacheHelperProportions();
@@ -346,11 +346,11 @@ $.widget("ui.draggable", $.ui.mouse, {
346346
_setHandleClassName: function() {
347347
this.handleElement = this.options.handle ?
348348
this.element.find( this.options.handle ) : this.element;
349-
this.handleElement.addClass( "ui-draggable-handle" );
349+
this._addClass( this.handleElement, "ui-draggable-handle" );
350350
},
351351

352352
_removeHandleClassName: function() {
353-
this.handleElement.removeClass( "ui-draggable-handle" );
353+
this._removeClass( this.handleElement, "ui-draggable-handle" );
354354
},
355355

356356
_createHelper: function(event) {
@@ -654,7 +654,7 @@ $.widget("ui.draggable", $.ui.mouse, {
654654
},
655655

656656
_clear: function() {
657-
this.helper.removeClass("ui-draggable-dragging");
657+
this._removeClass( this.helper, "ui-draggable-dragging" );
658658
if (this.helper[0] !== this.element[0] && !this.cancelHelperRemoval) {
659659
this.helper.remove();
660660
}

0 commit comments

Comments
 (0)