Details
-
Type:
Improvement
-
Status:
Open
-
Priority:
Minor
-
Resolution: Unresolved
-
Affects Version/s: None
-
Fix Version/s: None
-
Component/s: Web - General
-
Labels:None
Description
The Google toolbar adds a handy form completion feature to browsers. It highlights form fields that can be filled with a single click. Alas, the HTML forms generated by signup.jsp and userForm.jsp don't work so well with this plugin. The Google feature put a city name in the State field and failed to fill out the country. I was able to improve this by simply consistently placing the label before the input field. See the diff for userForm.jsp below. I also renamed the text for user.address.address=Street to avoid an insipid looking repetition of the word Address in the form.
@@ -119,28 +119,28 @@
<label class="desc"><fmt:message key="user.address.address"/></label>
<div class="group">
<div>
+ <p><appfuse:label key="user.address.address"/></p>
<form:input path="address.address" id="address.address" cssClass="text large"/>
<form:errors path="address.address" cssClass="fieldError"/>
- <p><appfuse:label key="user.address.address"/></p>
</div>
<div class="left">
+ <p><appfuse:label key="user.address.city"/></p>
<form:input path="address.city" id="address.city" cssClass="text medium"/>
<form:errors path="address.city" cssClass="fieldError"/> - <p><appfuse:label key="user.address.city"/></p>
</div>
<div>
+ <p><appfuse:label key="user.address.province"/></p>
<form:input path="address.province" id="address.province" cssClass="text state" size="2"/>
<form:errors path="address.province" cssClass="fieldError"/> - <p><appfuse:label key="user.address.province"/></p>
</div>
<div class="left">
+ <p><appfuse:label key="user.address.postalCode"/></p>
<form:input path="address.postalCode" id="address.postalCode" cssClass="text zip"/>
<form:errors path="address.postalCode" cssClass="fieldError"/> - <p><appfuse:label key="user.address.postalCode"/></p>
</div>
<div> - <appfuse:country name="address.country" prompt="" default="$
{user.address.country}"/>
<p><appfuse:label key="user.address.country"/></p>
+ <appfuse:country name="address.country" prompt="" default="${user.address.country}"/>
</div>
</div>
</li>