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