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 2011/08/12 19:31:22 UTC
svn commit: r1157193 -
/tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js
Author: hlship
Date: Fri Aug 12 17:31:22 2011
New Revision: 1157193
URL: http://svn.apache.org/viewvc?rev=1157193&view=rev
Log:
TAP5-877: Fine tune some adjustments for IE support of client-side validation error bubble
Modified:
tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js
Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js?rev=1157193&r1=1157192&r2=1157193&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js Fri Aug 12 17:31:22 2011
@@ -1295,6 +1295,8 @@ Tapestry.ErrorPopup = Class.create({
BUBBLE_HEIGHT : "39px",
+ IE_FADE_TIME : 500,
+
initialize : function(field) {
this.field = $(field);
@@ -1344,7 +1346,7 @@ Tapestry.ErrorPopup = Class.create({
/*
* If this field is not the focus field after a focus change, then
- * it's bubble, if visible, should fade out. This covers tabbing
+ * its bubble, if visible, should fade out. This covers tabbing
* from one form to another.
*/
this.fadeOut();
@@ -1387,7 +1389,7 @@ Tapestry.ErrorPopup = Class.create({
var bound = _.bind(this.hideIfNotFocused, this);
- _.delay(bound, 100);
+ _.delay(bound, this.IE_FADE_TIME);
return;
}
@@ -1423,8 +1425,12 @@ Tapestry.ErrorPopup = Class.create({
if (this.animation)
return;
- if (Prototype.IE) {
- this.outerDiv.hide();
+ if (Prototype.Browser.IE) {
+
+ var div = this.outerDiv;
+
+ _.delay(function() { div.hide(); }, this.IE_FADE_TIME);
+
return;
}