> I didn't remove the language attribute in case some browsers need it (I
> don't know of any though).

On that note, do we have an official list of supported browsers and/or
versions that we can actually test against?  You know, for browser issues
(e.g. JavaScript, HTML tags, XHTML, etc, etc.)

Sorry if I missed that somewhere.



James Mitchell
Software Engineer/Struts Evangelist
http://www.open-tools.org




> -----Original Message-----
> From: [EMAIL PROTECTED] [mailto:dgraham@;apache.org]
> Sent: Sunday, October 20, 2002 7:17 PM
> To: [EMAIL PROTECTED]
> Subject: cvs commit:
> jakarta-struts/src/share/org/apache/struts/taglib/html
> JavascriptValidatorTag.java
>
>
> dgraham     2002/10/20 16:17:06
>
>   Modified:    src/share/org/apache/struts/taglib/html
>                         JavascriptValidatorTag.java
>   Log:
>   Added type="text/javascript" to script tags for xhtml
> compliance.  I didn't
>   remove the language attribute in case some browsers need it (I
> don't know of
>   any though).
>
>   Revision  Changes    Path
>   1.9       +3 -3
> jakarta-struts/src/share/org/apache/struts/taglib/html/JavascriptV
> alidatorTag.java
>
>   Index: JavascriptValidatorTag.java
>   ===================================================================
>   RCS file:
> /home/cvs/jakarta-struts/src/share/org/apache/struts/taglib/html/J
> avascriptValidatorTag.java,v
>   retrieving revision 1.8
>   retrieving revision 1.9
>   diff -u -r1.8 -r1.9
>   --- JavascriptValidatorTag.java     18 Oct 2002 16:58:10 -0000      1.8
>   +++ JavascriptValidatorTag.java     20 Oct 2002 23:17:06 -0000      1.9
>   @@ -394,7 +394,7 @@
>                    results.append("    } \n\n");
>                }
>            } else if ("true".equals(staticJavascript)) {
>   -            results.append("<script language=\"Javascript1.1\">");
>   +            results.append("<script language=\"Javascript1.1\"
> type=\"text/javascript\">");
>            }
>
>            if ("true".equals(staticJavascript)) {
>   @@ -438,7 +438,7 @@
>            StringBuffer sb = new StringBuffer();
>            String name = formName.substring(0, 1).toUpperCase() +
> formName.substring(1, formName.length());
>
>   -        sb.append("<script language=\"Javascript1.1\"");
>   +        sb.append("<script language=\"Javascript1.1\"
> type=\"text/javascript\"");
>
>            if (src != null) {
>                sb.append(" src=\"" + src + "\"> \n");
>
>
>
>
> --
> To unsubscribe, e-mail:
> <mailto:struts-dev-unsubscribe@;jakarta.apache.org>
> For additional commands, e-mail:
> <mailto:struts-dev-help@;jakarta.apache.org>
>
>


--
To unsubscribe, e-mail:   <mailto:struts-dev-unsubscribe@;jakarta.apache.org>
For additional commands, e-mail: <mailto:struts-dev-help@;jakarta.apache.org>

Reply via email to