On 8/19/06, Martijn Dashorst <[EMAIL PROTECTED]> wrote:
> Coming in late for the party, but now you see the benefits of
> mailinglists instead of IRC :-)

And I was just about to propose to shut down the lists in favor of the
IRC channel. Or wait...

> I agree with Igor, in the sense that we should be careful with adding
> convenience stuff to core. On the other hand we try to create a
> workable framework that actually saves people time.
>
> I think this could be a perfect example for the Wiki, or an example
> document on our front page, showing how *you* can expand the Wicket
> framework.

Ok, so say we make that class in the examples, and we use it
everywhere where we now use WebMarkupContainer. I almost smell those
messages pouring in like: 'where can I find that component? Why isn't
it part of core?'.

I still feel that of all the convenience we can think of, this is the
lightest offender and is actually helpful in the way that it gives a
better name for something people will do a lot. And even if they can
easily create such a component themselves, they probably don't, just
like none of use haven't for any of the comparable cases.

Maybe I should put this differently, or start a vote: who is really
against this?

Eelco

>
> Martijn
>
> On 8/15/06, Eelco Hillenius <[EMAIL PROTECTED]> wrote:
> > On 8/14/06, Igor Vaynberg <[EMAIL PROTECTED]> wrote:
> > > i think 1 is trivial and should not be part of the core. our api has a
> > > pretty large surface for the users to learn already and i dont think we 
> > > need
> > > to throw such things in.
> >
> > I would usually agree with that, but in this case I'd be +1 for the
> > addition. TagModifier looks a lot better to me than
> > WebMarkupContainer, the latter being a base class, and TagModifier's
> > name only implying it is for a specific purpose. *If* we would agree
> > to put this in the project, we state in it's API docs that it is not
> > mean for anything else.
> >
> > Eelco

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Wicket-develop mailing list
Wicket-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-develop

Reply via email to