Skip to content
This repository has been archived by the owner on Oct 8, 2021. It is now read-only.

Commit

Permalink
Dialog: Removed code that removes active state from buttons on pagehi…
Browse files Browse the repository at this point in the history
…de. Fixes #5111.

The code was added because close buttons still had active state when you closed and re-opened a dialog (#1839). I tested without the code and that issue doesn't occur anymore (nav takes care of removing active state).
  • Loading branch information
jaspermdegroot committed Jan 3, 2013
1 parent aded724 commit 4d60cd2
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions js/widgets/dialog.js
Expand Up @@ -59,9 +59,6 @@ $.widget( "mobile.dialog", $.mobile.widget, {
$target.attr( "data-" + $.mobile.ns + "transition", ( active.transition || $.mobile.defaultDialogTransition ) )
.attr( "data-" + $.mobile.ns + "direction", "reverse" );
}
})
.bind( "pagehide", function( e, ui ) {
$( this ).find( "." + $.mobile.activeBtnClass ).not( ".ui-slider-bg" ).removeClass( $.mobile.activeBtnClass );
});

this._on( $el, {
Expand Down

0 comments on commit 4d60cd2

Please sign in to comment.