From d22c8c0bb413f40ebf0a6063c21b3bdcbcbb7af2 Mon Sep 17 00:00:00 2001 From: Dan Helfman Date: Tue, 3 Jun 2008 16:12:13 -0700 Subject: [PATCH] Fixed bug where form onsubmit event binding was only being done on the last form in a note. --- static/js/Editor.js | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/static/js/Editor.js b/static/js/Editor.js index ba36ff0..1979557 100644 --- a/static/js/Editor.js +++ b/static/js/Editor.js @@ -156,15 +156,19 @@ Editor.prototype.finish_init = function () { connect( this.document, "onclick", function ( event ) { self.mouse_clicked( event ); } ); // handle each form submit event by forwarding it on as a custom event - var forms = getElementsByTagAndClassName( "form", null, this.document ); - for ( var i in forms ) { - var form = forms[ i ]; + function connect_form( form ) { connect( form, "onsubmit", function ( event ) { signal( self, "submit_form", form ); event.stop(); } ); } + var forms = getElementsByTagAndClassName( "form", null, this.document ); + for ( var i in forms ) { + var form = forms[ i ]; + connect_form( form ); + } + // connect each (non-submit) button to issue an event function connect_button( button ) { connect( button, "onclick", function ( event ) {