[jsr-314-open] composite:insertFacet target facet name

Alexander Smirnov asmirnov at exadel.com
Fri Sep 18 18:44:37 EDT 2009



On 09/18/2009 01:46 PM, Ken Paulsen wrote:
>
> When / where should this discussion take place? Do we want to have a
> call for this?
>
> In addition to this issue, Alexander raised the issue that f:insertFacet
> and f:renderFacet are confusing. Not sure if there's anything we can do
> resolve this at this point, but minimally, it would be worth ensuring
> the EG members understand the difference (which IMO, is huge). In
> hindsight, f:insertFacet probably should have been f:attachFacet, and of
> course had an optional "target" attribute for Andy's case.
My point was that
<f:facet name="caption">
    <composite:renderFacet name="caption" />
</f:facet>
and
<composite:insertFacet name="caption" />
have to have exact same functionality ( although they may not do it now 
) that will make 'insrtFacet' unnecessary duplicate.
But, at least, I agree that limitation that requires to have composite 
facet name the same as target should be resolved, not only to have 
ability to insert facet into different components but also to allow 
component developer give more informative names for facets ( for 
example, something like "userDescription" or "errorMessages" instead of 
"caption" for some special components ).
>
> Ken
>
> Andy Schwartz wrote:
>> Thanks Ed -
>>
>> Ed Burns wrote:
>>>
>>> I happen to prefer #1, but everyone else favors #2, we'll go with #2.
>>
>> Seems like some people prefer #2 as well, so perhaps this needs more
>> discussion.
>>
>>> 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.
>>>
>>
>> Sorry for taking so long to follow up on this. I have logged the
>> following spec issue:
>>
>> https://javaserverfaces-spec-public.dev.java.net/issues/show_bug.cgi?id=631
>>
>>
>> Andy
>>
>>




More information about the jsr-314-open-mirror mailing list