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


Reply via email to