You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by Hazem Saleh <ha...@apache.org> on 2008/07/02 00:01:02 UTC

Re: [VOTE] promoting the selectOneRow component to Tomahawk

Congratulations!
Promotion is done.

On Mon, Jun 30, 2008 at 8:06 PM, Leonardo Uribe <lu...@gmail.com> wrote:

> +1
>
>
> On Mon, Jun 30, 2008 at 1:26 AM, Cagatay Civici <ca...@gmail.com>
> wrote:
>
>> +1 for the promotion, as I remember this component was donated to myfaces
>> in 2005 by my brother, and I think 3 years is "enough" to live in sandbox.
>>
>> Cagatay.
>>
>>
>> On Fri, Jun 27, 2008 at 9:44 PM, Mike Kienenberger <mk...@gmail.com>
>> wrote:
>>
>>> If I understand correctly, the current state of the component is such
>>> that it works with any UIData component.   I'm +1 for keeping that
>>> status.   It'd be great if we could do something similar with many of
>>> the other t:dataTable functionality.
>>>
>>> On 6/27/08, simon <si...@chello.at> wrote:
>>> > I'm +1 to promoting the component in its current state (which I think
>>> is
>>> >  your -1 option below).
>>> >
>>> >  Thanks for looking into that.
>>> >
>>> >  Regards,
>>> >
>>> > Simon
>>> >
>>> >
>>> >  On Fri, 2008-06-27 at 15:24 +0300, Hazem Saleh wrote:
>>> >  > Hi Simon,
>>> >  >
>>> >  > I found it possible to use it with the h:dataTable as it depends
>>> only
>>> >  > on (UIData).
>>> >  > We have now two paths:
>>> >  > [+1] for merging this component as an attribute in <t:dataTable
>>> .../>.
>>> >  > [-1] for promoting the component to Tomahawk in its current state.
>>> >  >
>>> >  > Me >> -1 now.
>>> >  >
>>> >  > On Mon, Jun 16, 2008 at 10:04 AM, simon.kitching@chello.at
>>> >  > <si...@chello.at> wrote:
>>> >  >         I think it would be useful to have an answer to my question
>>> >  >         first.
>>> >  >
>>> >  >         *Is* it possible to do this?
>>> >  >          <h:dataTable ...>
>>> >  >            <s:selectOneRow/>
>>> >  >          </h:dataTable>
>>> >  >
>>> >  >         If so, then moving from the current approach to using an
>>> >  >         attribute will
>>> >  >         remove existing functionality. Ok, with a sandbox component
>>> >  >         that is
>>> >  >         allowed but the above looks useful.
>>> >  >
>>> >  >         Unfortunately I'm really busy on other things at the moment,
>>> >  >         and don't
>>> >  >         have time to look myself.
>>> >  >
>>> >  >         Regards,
>>> >  >         Simon
>>> >  >
>>> >  >         Hazem Saleh schrieb:
>>> >  >         > Any new votes ?
>>> >  >         >
>>> >  >         > On Sun, Jun 15, 2008 at 4:13 PM, simon
>>> >  >         <simon.kitching@chello.at
>>> >  >
>>> >  >         > <ma...@chello.at>> wrote:
>>> >  >         >
>>> >  >         >     I only suggested leaving it as it is *if* it is
>>> possible
>>> >  >         to use this
>>> >  >         >     component with tables other than t:dataTable.
>>> >  >         >
>>> >  >         >     On Sun, 2008-06-15 at 15:57 +0300, Hazem Saleh wrote:
>>> >  >         >     > OK Let's make this as a vote :
>>> >  >         >     > we now have :
>>> >  >         >     > -> 2 votes for supporting the feature inside the
>>> >  >         t:dataTable.
>>> >  >         >     > -> 1 vote for leaving the current component as is.
>>> >  >         >     >
>>> >  >         >     > On Sun, Jun 15, 2008 at 3:13 PM, simon
>>> >  >         <simon.kitching@chello.at
>>> >  >
>>> >  >         >     <ma...@chello.at>>
>>> >  >         >     > wrote:
>>> >  >         >     >         A separate component would be nice if it
>>> could
>>> >  >         be applied to
>>> >  >         >     >         any table,
>>> >  >         >     >         eg the h:dataTable.
>>> >  >         >     >
>>> >  >         >     >
>>> >  >         >     >         On Sun, 2008-06-15 at 13:40 +0300, Hazem
>>> Saleh
>>> >  >         wrote:
>>> >  >         >     >         > +1 for Matzew idea.
>>> >  >         >     >         > I think that the old component syntax was
>>> >  >         suitable for the
>>> >  >         >     >         component
>>> >  >         >     >         > when it was in the sandbox phase.
>>> >  >         >     >         > I will implement this idea, it is good.
>>> >  >         >     >         > Thanks Matzew!
>>> >  >         >     >         >
>>> >  >         >     >         > On Sun, Jun 15, 2008 at 11:16 AM, Matthias
>>> >  >         Wessendorf
>>> >  >
>>> >  >         >     >         > <matzew@apache.org
>>> >  >         <ma...@apache.org>> wrote:
>>> >  >         >     >         >         On Sat, Jun 14, 2008 at 6:16 PM,
>>> >  >         Hazem Saleh
>>> >  >
>>> >  >         >     >         >         <hazems@apache.org
>>> >  >         <ma...@apache.org>>
>>> >  >
>>> >  >         >     wrote:
>>> >  >         >     >         >         > Hi Team,
>>> >  >         >     >         >         >
>>> >  >         >     >         >         > After updating the selectOneRow
>>> >  >         component's
>>> >  >         >     >         documentation.
>>> >  >         >     >         >
>>> >  >         >     >         >
>>> >  >         >     >         >         just curious, what is this
>>> component
>>> >  >         about?
>>> >  >         >     >         >         Using it like this ?
>>> >  >         >     >         >
>>> >  >         >     >         >         <t:table .....>
>>> >  >         >     >         >          <s:selectOneRow />
>>> >  >         >     >         >         </t:able>
>>> >  >         >     >         >
>>> >  >         >     >         >         If so, how to declare multiple row
>>> >  >         selection ?
>>> >  >         >     >         >
>>> >  >         >     >         >         Trinidad's table doesn't treat
>>> that
>>> >  >         as a special
>>> >  >         >     >         case. It just
>>> >  >         >     >         >         has an
>>> >  >         >     >         >         attribute for that, where you
>>> >  >         specify via an enum
>>> >  >         >     >         the
>>> >  >         >     >         >         selection type
>>> >  >         >     >         >         (single:mulit)
>>> >  >         >     >         >
>>> >  >         >     >         >         Greetings,
>>> >  >         >     >         >         Matthias
>>> >  >         >     >         >
>>> >  >         >     >         >
>>> >  >         >     >         >         >
>>> >  >         >     >         >         > I wish to promote it to the next
>>> >  >         Tomahawk
>>> >  >         >     release.
>>> >  >         >     >         >         >
>>> >  >         >     >         >         > [+1] for agreeing with promoting
>>> >  >         the
>>> >  >         >     component to
>>> >  >         >     >         the next
>>> >  >         >     >         >         Tomahawk release.
>>> >  >         >     >         >         > [-1] for disagreeing with
>>> >  >         promoting the
>>> >  >         >     component
>>> >  >         >     >         to the
>>> >  >         >     >         >         next Tomahawk
>>> >  >         >     >         >         > release.
>>> >  >         >
>>> >  >
>>> >  >
>>> >  >
>>> >  >
>>> >  >
>>> >  > --
>>> >  > Hazem Ahmed Saleh Ahmed
>>> >  > http://www.jroller.com/page/HazemBlog
>>> >
>>> >
>>>
>>
>>
>


-- 
Hazem Ahmed Saleh Ahmed
http://www.jroller.com/page/HazemBlog