Dialog: Don't use .attr( props, true ).

diff --git a/ui/jquery.ui.dialog.js b/ui/jquery.ui.dialog.js
index dcf44580ef1ca14644031c495685409a7e6593cf..203618fc25dcd2f92825ae3a6b0bfed659988f57 100644 (file)
--- a/ui/jquery.ui.dialog.js
+++ b/ui/jquery.ui.dialog.js
@@ -334,15 +334,18 @@ $.widget("ui.dialog", {
                }
                if ( hasButtons ) {
                        $.each( buttons, function( name, props ) {
+                               var button, click;
                                props = $.isFunction( props ) ?
                                        { click: props, text: name } :
                                        props;
-                               var button = $( "<button type='button'></button>" )
-                                       .attr( props, true )
-                                       .unbind( "click" )
-                                       .click(function() {
-                                               props.click.apply( that.element[0], arguments );
-                                       })
+                               // Default to a non-submitting button
+                               props = $.extend( { type: "button" }, props );
+                               // Change the context for the click callback to be the main element
+                               click = props.click;
+                               props.click = function() {
+                                       click.apply( that.element[0], arguments );
+                               };
+                               button = $( "<button></button>", props )
                                        .appendTo( that.uiButtonSet );
                                if ( $.fn.button ) {
                                        button.button();