Skip to content

Commit dffc5f5

Browse files
committed
Dialog: Remove busted ui-dialog-disabled class, shouldn't be there. Removed outdated TODOs.
1 parent 552f799 commit dffc5f5

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

ui/jquery.ui.dialog.js

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -427,7 +427,6 @@ $.widget("ui.dialog", {
427427
});
428428
},
429429

430-
// TODO why are handles passed by _setOption??
431430
_makeResizable: function() {
432431
var that = this,
433432
options = this.options,
@@ -580,15 +579,6 @@ $.widget("ui.dialog", {
580579
});
581580
}
582581

583-
if ( key === "disabled" ) {
584-
// TODO use toggleClass( "ui-dialog-disabled", value )
585-
if ( value ) {
586-
uiDialog.addClass( "ui-dialog-disabled" );
587-
} else {
588-
uiDialog.removeClass( "ui-dialog-disabled" );
589-
}
590-
}
591-
592582
if ( key === "draggable" ) {
593583
isDraggable = uiDialog.is( ":data(ui-draggable)" );
594584
if ( isDraggable && !value ) {
@@ -624,7 +614,6 @@ $.widget("ui.dialog", {
624614

625615
if ( key === "title" ) {
626616
// convert whatever was passed in to a string, for html() to not throw up
627-
// TODO deduplicate this (see _create)
628617
$( ".ui-dialog-title", this.uiDialogTitlebar )
629618
.html( "" + ( value || " " ) );
630619
}

0 commit comments

Comments
 (0)