Re: [jsr-314-open] composite:insertFacet target facet name
On 09/16/2009 10:59 AM, Ed Burns wrote:
>>>>> On Tue, 15 Sep 2009 19:29:22 -0400, Andy
Schwartz<andy.schwartz(a)oracle.com> said:
AS> Gang -
AS> As currently specified, composite:insertFacet's "name" attribute
serves
AS> two purposes:
AS> 1. It identifies the name of the facet on the containing composite
AS> component to insert.
AS> 2. It identifies the name of the facet on the target component into
AS> which the facet is being inserted.
[...]
AS> 1. Add an attribute to composite:insertFacet that allows a target facet
AS> name to be specified:
AS> <h:panelGrid>
AS> <composite:insertFacet name="backupCaption"
targetName="caption"/>
AS> </h:panelGrid>
AS> 2. Specify that the target facet name can be picked up from a wrapping
AS> <f:facet> tag:
AS> <h:panelGrid>
AS> <f:facet name="caption">
AS> <composite:insertFacet name="backupCaption"/>
AS> </f:facet>
AS> </h:panelGrid>
AS> I prefer #2 since is consistent with typical facet usage.
I happen to prefer #1, but everyone else favors #2, we'll go with #2.
Sorry for
wrong decigion, I've mixed 'composite:insertFacet' and
'composite:renderFacet' tags. The second option is makes sense for
'renderFacet' but 'insertFacet' seems better to use an additional
attribute.
PS. I really don't see an advantage to have two so alike tags, except
performance purposes. As you already saw even EG members are messed up
with them :-)
Andy, can you please file a spec issue and share the number with the
group? Once you have it, I'll add an entry in the changelog wiki.
Ed