[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-1598) include "Clear" button for seam-gen search form

[Dan Allen (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBSEAM-1598?page=comments#action_12368815 ] 
            
Dan Allen commented on JBSEAM-1598:
-----------------------------------

I think you nailed it right on the head. This is another one of those "oops, they forgot to add this to JSF" type of issues. It is very similar in nature to the s:link. A <input type="reset" /> is useless because it just restores the form to the original state. What you really want is to remove all values from a form and submit. I would lean towards s:clearButton and s:clearLink rather than rich since it really is core stuff.

I argue that you do want to call and action (potentially) because you might need a method to refetch the list.  Something like:

<s:clearLink action="#{itemList.query}" />

Of course, if you are going with all page parameters, then you can just do

<s:clearLink action="/itemList.xhtml" />

> include "Clear" button for seam-gen search form
> -----------------------------------------------
>
>                 Key: JBSEAM-1598
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-1598
>             Project: JBoss Seam
>          Issue Type: Feature Request
>          Components: Tools
>    Affects Versions: 2.0.0.BETA1
>            Reporter: Dan Allen
>            Priority: Minor
>   Original Estimate: 30 minutes
>  Remaining Estimate: 30 minutes
>
> The search is a very nice feature to have on the list pages generated by seam-gen, but I have watched people (and struggled myself) with figuring out how to clear the current  filter.  It is very annoying to have to manually erase all the values entered in the form and then submit the form again once cleared.  Far easier would be to have a Clear button on this form.
> The following button would do the trick.
> <h:commandButton id="clear" value="Clear" onclick="for (var i = 0, len = this.form.length; i &lt; len; i++) { if (this.form[i].type == 'text') { this.form[i].value = ''; } }" action="/${listPageName}.xhtml"/>
> I will admit that the JavaScript is a little hacky, but it gets the point across.  Feel free to refine.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira