Skip to content

Commit

Permalink
Popup: Use duration:fast for default animations. Also fix two missing…
Browse files Browse the repository at this point in the history
… semicolons.
  • Loading branch information
jzaefferer committed Sep 21, 2011
1 parent 2196b74 commit a3866bf
Showing 1 changed file with 11 additions and 5 deletions.
16 changes: 11 additions & 5 deletions ui/jquery.ui.popup.js
Expand Up @@ -23,8 +23,14 @@ $.widget( "ui.popup", {
my: "left top",
at: "left bottom"
},
show: "slideDown",
hide: "fadeOut"
show: {
effect: "slideDown",
duration: "fast"
},
hide: {
effect: "fadeOut",
duration: "fast"
}
},
_create: function() {
if ( !this.options.trigger ) {
Expand All @@ -47,7 +53,7 @@ $.widget( "ui.popup", {
.attr( "aria-owns", this.element.attr( "id" ) );

this.element
.addClass( "ui-popup" )
.addClass( "ui-popup" );
this._beforeClose();
this.element.hide();

Expand Down Expand Up @@ -134,7 +140,7 @@ $.widget( "ui.popup", {
this.close( event );
}
}
})
});
},

_destroy: function() {
Expand Down Expand Up @@ -203,7 +209,7 @@ $.widget( "ui.popup", {
this.isOpen = false;
this._trigger( "close", event );
},

_beforeClose: function() {
this.element
.attr( "aria-hidden", "true" )
Expand Down

0 comments on commit a3866bf

Please sign in to comment.