You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@myfaces.apache.org by Mike Kienenberger <mk...@gmail.com> on 2006/01/02 19:12:51 UTC

Re: Why do h:message Tags Need to be enclosed in h:panelGroup or h:panelGrid Tags

Mike and Gert,

I've modified MYFACES-993 -- t:div, t:span, s:fieldset should render
children -- to handle this issue as it's directly related to t:div
rendering its children.  I didn't see another issue opened on this
topic yet.

This would be a good place to attach patches to fix this issue.


On 12/21/05, Gert Vanthienen <ge...@skynet.be> wrote:
> Mike,
>
> What would be the correct way to make this happen in MyFaces?  Add an
> issue to JIRA and attach the solution for <s:fieldset/> to it?
>
> Gert
>
>
> Mike Duffy wrote:
> > 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 <ge...@skynet.be> 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
> >>
> >>
>
>