// Ideally we'd use e.target instead of $historySubmitter, but e.target points
// to the form element for submit actions, so.
// Ideally we'd use e.target instead of $historySubmitter, but e.target points
// to the form element for submit actions, so.
// Without the cloning we'd be changing the real form, which is slower, could make
// the page look broken for a second in slow browsers and might show the form broken
// again when coming back from a "next" page.
// Without the cloning we'd be changing the real form, which is slower, could make
// the page look broken for a second in slow browsers and might show the form broken
// again when coming back from a "next" page.
- $historyCompareForm.submit( function ( e ) {
- var $copyForm, $copyRadios, $copyAction;
+ $historyCompareForm.on( 'submit', function ( e ) {
+ var $copyForm, $copyRadios, $copyAction;
// Also remove potentially conflicting id attributes that we don't need anyway
$copyForm
.css( 'display', 'none' )
.find( '[id]' ).removeAttr( 'id' )
.end()
.insertAfter( $historyCompareForm )
// Also remove potentially conflicting id attributes that we don't need anyway
$copyForm
.css( 'display', 'none' )
.find( '[id]' ).removeAttr( 'id' )
.end()
.insertAfter( $historyCompareForm )