You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by jc...@apache.org on 2007/06/04 13:43:18 UTC

svn commit: r544128 - /incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js

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));