You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by mr...@apache.org on 2006/04/28 06:58:35 UTC

svn commit: r397759 - /struts/action/trunk/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java

Author: mrdon
Date: Thu Apr 27 21:58:33 2006
New Revision: 397759

URL: http://svn.apache.org/viewcvs?rev=397759&view=rev
Log:
Adding check for visibility before giving field focus, applying patch by
Zarar Siddiqi 
STR-2826

Modified:
    struts/action/trunk/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java

Modified: struts/action/trunk/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java
URL: http://svn.apache.org/viewcvs/struts/action/trunk/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java?rev=397759&r1=397758&r2=397759&view=diff
==============================================================================
--- struts/action/trunk/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java (original)
+++ struts/action/trunk/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java Thu Apr 27 21:58:33 2006
@@ -659,8 +659,9 @@
         results.append(lineEnd);
         results.append(lineEnd);
 
-        results.append(
-            "  if (focusControl.type != \"hidden\" && !focusControl.disabled) {");
+        results.append("  if (focusControl.type != \"hidden\" && ");
+        results.append("!focusControl.disabled && ");
+        results.append("focusControl.style.display != \"none\") {");
         results.append(lineEnd);
 
         // Construct the index if needed and insert into focus statement



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org