Author: jcompagner Date: Mon Jun 4 04:43:17 2007 New Revision: 544128 URL: http://svn.apache.org/viewvc?view=rev&rev=544128 Log: if precondition returns false then still call done focus problems in IE fixed
Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js?view=diff&rev=544128&r1=544127&r2=544128 ============================================================================== --- incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js (original) +++ incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js Mon Jun 4 04:43:17 2007 @@ -687,7 +687,8 @@ return false; } } else { - return null; + this.done(); + return false; } }, @@ -725,7 +726,8 @@ return false; } } else { - return null; + this.done(); + return false; } }, @@ -917,11 +919,13 @@ // This should be changed for IE See comments in wicket-event.js add (attachEvent/detachEvent) // IE this will cause double events for everything.. (mostly because of the Function.prototype.bind(element)) Wicket.Focus.attachFocusEvent(); - // set the focus to the last component - Wicket.Focus.requestFocus(); this.request.done(); this.successHandler(); + + // set the focus to the last component + setTimeout("Wicket.Focus.requestFocus();", 0); + // continue to next step (which should make the processing stop, as success should be the final step) notify(); }.bind(this));