diff options
| author | Jörn Zaefferer <joern.zaefferer@gmail.com> | 2012-11-09 17:17:32 +0100 |
|---|---|---|
| committer | Jörn Zaefferer <joern.zaefferer@gmail.com> | 2012-11-26 10:26:11 +0100 |
| commit | 46327804350874d21c564bc72d7d3c541bef9378 (patch) | |
| tree | 87afbe89aa2abd55d4b178dea69626a5c52f6b2c /ui/jquery.ui.dialog.js | |
| parent | 2062a18db6eb1544c2f28d68b6f55d60eef0fd65 (diff) | |
| download | jquery-ui-46327804350874d21c564bc72d7d3c541bef9378.tar.gz jquery-ui-46327804350874d21c564bc72d7d3c541bef9378.zip | |
Dialog: Fix yoda-if, remove unnecessary TODOs; add missing callbacks to commons test
Diffstat (limited to 'ui/jquery.ui.dialog.js')
| -rw-r--r-- | ui/jquery.ui.dialog.js | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/ui/jquery.ui.dialog.js b/ui/jquery.ui.dialog.js index 7545a8adf..04b23f2a1 100644 --- a/ui/jquery.ui.dialog.js +++ b/ui/jquery.ui.dialog.js @@ -94,7 +94,6 @@ $.widget("ui.dialog", { parent: this.element.parent(), index: this.element.parent().children().index( this.element ) }; - // TODO don't overwrite options this.options.title = this.options.title || this.originalTitle; var that = this, options = this.options, @@ -276,8 +275,7 @@ $.widget("ui.dialog", { return; } - // TODO fix yoda-if - if ( false === this._trigger( "beforeClose", event ) ) { + if ( this._trigger( "beforeClose", event ) === false ) { return; } @@ -294,7 +292,6 @@ $.widget("ui.dialog", { $( this.document[ 0 ].activeElement ).blur(); } - // TODO shouldn't _hide restore `this` to the instance? would also help tooltip this._hide( this.uiDialog, this.options.hide, function() { that._trigger( "close", event ); }); |