You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by hl...@apache.org on 2008/04/02 00:29:30 UTC

svn commit: r643627 - /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry/tapestry.js

Author: hlship
Date: Tue Apr  1 15:29:25 2008
New Revision: 643627

URL: http://svn.apache.org/viewvc?rev=643627&view=rev
Log:
TAPESTRY-2320: Problem with validations popups on Internet Explorer 7

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry/tapestry.js

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry/tapestry.js
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry/tapestry.js?rev=643627&r1=643626&r2=643627&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry/tapestry.js (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry/tapestry.js Tue Apr  1 15:29:25 2008
@@ -176,7 +176,7 @@
 
                 element.request({ onSuccess : successHandler });
 
-                event.stop();
+                Event.stop(event); // Should be domevent.stop(), but that fails under IE
 
                 return false;
             };
@@ -438,7 +438,9 @@
         this.innerSpan = new Element("span");
         this.outerDiv = $(new Element("div", { 'class' : 't-error-popup' })).update(this.innerSpan).hide();
 
-        this.field.insert({ after : this.outerDiv });
+        var body = $$('body')[0];
+
+        body.insert({ bottom: this.outerDiv });
 
         this.outerDiv.absolutize();
 
@@ -450,7 +452,7 @@
 
             this.field.focus();
 
-            event.stop();
+            Event.stop(event);  // Should be domevent.stop(), but that fails under IE
         }.bindAsEventListener(this));
 
         Tapestry.errorPopups.push(this);
@@ -590,7 +592,7 @@
 
         if (! event.result)
         {
-            domevent.stop();
+            Event.stop(domevent); // Should be domevent.stop(), but that fails under IE
         }
         else
         {