I am using the following JSP tag to render a <tr> and my label/text
field.

<ww:textfield label="getText('user.firstName')"
        name="'user.firstName'" value="user.firstName"/>

This results in the following XHTML:

<tr>
             <td align="right" valign="top">

<span class="label">

First Name:
                     </span>
         </td>

<td>

<input type="text"
                                    name="user.firstName"
/>
   </td>
</tr>

I know you can't do much about the spacing b/c these tags are backed
by Velocity templates. However, it would be more XHTML compliant (IMO) if there were no "align" attributes, a <th> was used, and a <label> for the <span>. For example:

<tr>
<th><label>First Name:</label></th>

Other improvements I'd like to see:

1. errors are displayed in a <span> after the <input>

2. <table> comes after <form> in <ww:form> tag.

3. <script> -> <script type="text/javascript">

3. form's cssStyle and cssClass get applied to <table> rather than <form>. They aren't applied to form right now anyway.

I'll attach a patch that fixes all of the above.