Wouldn't it be more logical to make this the default behaviour of
<s:fieldset/> (and arguably also for <t:div/>) in MyFaces, as their common 
usage in HTML also is
to group other elements?

Yes.  I think that is exactly correct.

Mike

--- Gert Vanthienen <[EMAIL PROTECTED]> wrote:

> L.S.,
> 
> Actually, it was quite easy to have the <s:fieldset/> render it's children,
> as described in the article referred to by Mike Kienenberger.  I downloaded
> the source code for the MyFaces Sandbox components and added these two
> methods to the FieldsetRenderer class:
> 
>     public boolean getRendersChildren() {
>         return true;
>     }
> 
>     public void encodeChildren(FacesContext context, UIComponent component)
> throws IOException {
>         RendererUtils.renderChildren(context, component);
>     }
> 
> Now, the <s:fieldset/> behaves like e.g. a panelGroup, allowing us to use it
> as a nesting container for other tags.
> 
> Wouldn't it be more logical to make this the default behaviour of
> <s:fieldset/> (and arguably also for <t:div/>) in MyFaces, as their common
> usage in HTML also is to group other elements?
> 
> Gert
> 
> -----Original Message-----
> From: Gert Vanthienen [mailto:[EMAIL PROTECTED] 
> Sent: maandag 19 december 2005 20:04
> To: 'MyFaces Discussion'
> Subject: RE: Why do h:message Tags Need to be enclosed in h:panelGroup or
> h:panelGrid Tags
> 
> Mike,
> 
> Not really...  I guess you could go and download the source code for MyFaces
> and create try to create your own div tag based on the example code from the
> current div-tag and the panelGroup tag.  I will be trying to do the same
> thing tomorrow with the <s:fieldset/> tag, I will let you know if it worked
> out ok...
> 
> Gert
> 
> -----Original Message-----
> From: Mike Duffy [mailto:[EMAIL PROTECTED] 
> Sent: donderdag 15 december 2005 21:18
> To: MyFaces Discussion
> Subject: RE: Why do h:message Tags Need to be enclosed in h:panelGroup or
> h:panelGrid Tags
> 
> Thx.
> 
> Any suggestions for a hack that could make this work?
> 
> --- Gert Vanthienen <[EMAIL PROTECTED]> wrote:
> 
> > L.S.,
> > 
> > 
> > I have the same problem when using a Sandbox <s:fieldset/> tag.  Wouldn't
> it
> > make sense to add this behaviour to both the fieldset and div tags (as
> they
> > are meant to group other UI elements anyway)?  
> > 
> > Another option could be to include a 'rendersChildren=true/false'
> attribute
> > to the tag definitions, so that the end user can decide whether or not the
> > div/fieldset is responsible for rendering it's children (because that's
> the
> > issue here, if I'm correct?)
> > 
> > 
> > Regards,
> > 
> > Gert Vanthienen
> > 
> > 
> > -----Original Message-----
> > From: Mike Duffy [mailto:[EMAIL PROTECTED] 
> > Sent: donderdag 15 december 2005 20:42
> > To: [email protected]
> > Subject: Why do h:message Tags Need to be enclosed in h:panelGroup or
> > h:panelGrid Tags
> > 
> > I am using the MyFaces <t:div> tag to layout my pages, which works fine.
> > However, because the
> > <h:message> tags are not contained in an h:panelGroup or h:panelGrid tag,
> > errors like the
> > following are generated when the page loads:
> > 
> > 00:01:52,097 ERROR [HtmlMessageRendererBase] Could not render Message.
> > Unable to find component
> > 'estimatedStartDate' (calling findComponent on component
> > 'issueCreateForm:estimatedStartDateError'). If the provided id was
> correct,
> > wrap the message and
> > its component into an h:panelGroup or h:panelGrid.
> > 
> > Why do h:message tags need to be enclosed in h:panelGroup or h:panelGrid
> > tags.
> > 
> > Does anyone else think this is a bug? Any suggestions?
> > 
> > Thx.
> > 
> > Mike
> > 
> > __________________________________________________
> > Do You Yahoo!?
> > Tired of spam?  Yahoo! Mail has the best spam protection around 
> > http://mail.yahoo.com 
> > 
> > -- 
> > No virus found in this incoming message.
> > Checked by AVG Free Edition.
> > Version: 7.1.371 / Virus Database: 267.13.13/199 - Release Date:
> 13-12-2005
> >  
> > 
> > -- 
> > No virus found in this outgoing message.
> > Checked by AVG Free Edition.
> > Version: 7.1.371 / Virus Database: 267.13.13/199 - Release Date:
> 13-12-2005
> >  
> > 
> > 
> 
> 
> __________________________________________________
> Do You Yahoo!?
> Tired of spam?  Yahoo! Mail has the best spam protection around 
> http://mail.yahoo.com 
> 
> -- 
> No virus found in this incoming message.
> Checked by AVG Free Edition.
> Version: 7.1.371 / Virus Database: 267.13.13/199 - Release Date: 13-12-2005
>  
> 
> -- 
> No virus found in this outgoing message.
> Checked by AVG Free Edition.
> Version: 7.1.371 / Virus Database: 267.14.1/206 - Release Date: 16-12-2005
>  
> 
> 
> -- 
> No virus found in this incoming message.
> Checked by AVG Free Edition.
> Version: 7.1.371 / Virus Database: 267.14.1/206 - Release Date: 16-12-2005
>  
> 
> -- 
> No virus found in this outgoing message.
> Checked by AVG Free Edition.
> Version: 7.1.371 / Virus Database: 267.14.1/207 - Release Date: 19-12-2005
>  
> 
> 


__________________________________________________
Do You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com 

Reply via email to