I don't know enough to comment on whether having getRendersChildren()
return true is a valid solution to this problem.
abstract boolean getRendersChildren()
Return a flag indicating whether this component is
responsible for rendering its child components.
I'm not sure when it's legal for a component to take responsibility
for rendering its children, and when it's not legal.
However, once that's been decided, the process would be to open a JIRA
issue, and attach a patch for s:fieldset and t:div.
On 12/21/05, Gert Vanthienen <[EMAIL PROTECTED]> 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 <[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
> >>
> >>
>
>
