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


Reply via email to