Re: Exporter new syntax discussion

+1 for the newer one.

On Thu, May 8, 2008 at 9:52 PM, Hazem Saleh <hazem.saleh@...> wrote:

Hi Gerald,

No special reason, I just asked people who use this component and they disliked the new syntax.

On Thu, May 8, 2008 at 9:29 PM, Gerald Müllan <gmuellan@...> wrote:

Hi,

i would prefer the new syntax since it seems to be more natural to me.

Similar to an updateActionListener; the command component fires the
listener embedded inside.

So, +1

Any special reason why breaking the common sense?

cheers,

Gerald

On Thu, May 8, 2008 at 8:01 PM, Hazem Saleh <hazem.saleh@...> wrote:

> Hi Team,
>
> After I started modifying of the Exporter component syntax,
>
> To be like this :
> <h:commandButton ...>
>         <s:exporter for="tbl_cars"
>               fileType="XLS"/>
>  </h:commandButton>
>
> Some guys didnot like this new syntax, and they preferred the current one :
> <s:exporter for="tbl_cars" fileType="XLS">
>     <h:commandButton .../>
> </s:exporter>
>
> I need your input regarding this.
> (+1) for supporting the new syntax.
> (-1) for supporting the old syntax.
>
> Thanks!
>
> --
> Hazem Ahmed Saleh Ahmed
> http://www.jroller.com/page/HazemBlog

--
http://www.irian.at

Your JSF powerhouse -
JSF Consulting, Development and
Courses in English and German

Professional Support for Apache MyFaces

--

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