[ 
https://issues.jboss.org/browse/SEAMFACES-27?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12603825#comment-12603825
 ] 

Christian Kaltepoth commented on SEAMFACES-27:
----------------------------------------------

Sure! Here it is:

https://github.com/seam/faces/pull/30

I tested the patch with a small sample application and everything looks fine. 
The only thing that could be discussed are the default values for the two new 
attributes. In my patch the default value for {{showGlobalMessages}} is 
{{true}} and {{showFieldMessages}} is {{false}}. I think this makes sense as 
{{<s:validateForm>}} will often be used for cross field validation.

Thoughts?

> UIValidateForm should allow control of where Validation Messages are displayed
> ------------------------------------------------------------------------------
>
>                 Key: SEAMFACES-27
>                 URL: https://issues.jboss.org/browse/SEAMFACES-27
>             Project: Seam Faces
>          Issue Type: Feature Request
>          Components: Validation & Conversion
>            Reporter: Lincoln Baxter III
>             Fix For: 3.0.2
>
>
> <s:validateForm validatorId="myValidator" showFieldMessages="true / false" 
> showGlobalMessages="true / false" />
> Perhaps issue a warning if both message targets are disabled.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        
_______________________________________________
seam-issues mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/seam-issues

Reply via email to