[Grok-dev] grokcore.viewlet ready to release

sylvain at infrae.com sylvain at infrae.com
Sun Nov 2 10:12:18 EST 2008


> Sylvain Viollon wrote:
>>    Does anyone have some time to just look at that extension and
>> check I didn't miss something obvious JW, Philipp ?
> Looking at it quickly, I have two remarks:
> 1) The order() directive always felt more generically useful than just
> for viewlets, even if it is still only used for viewlets. Would it be an
> idea to give this directive a more generic place? Or is it more or less
> easy to do that later in time when we actually have a use for the
> order() directive outside of viewlets?

  Well, I agree with you, but for the moment grokcore.viewlet is the only
component which use it. And, I don't think there is other components
than viewlets which use it in grok (maybe I am wrong), so we can safely
assume that it's the first best place to put that directive. The second
best place would be grokcore.component. After I don't see any usage of
order alone, at least for the moment.

  After we can still move that directive, yes, I don't see any problem to
that (or any problem which may need changes in user code  which use it).

> 2) I guess you want to "port" the Grok-trunk to make use of
> grokcore.viewlets asap?

  Yes, and merge my branch of five.grok which provides viewlets.

> (isn't it great to see even more packages split off of Grok!)

  Yes !

  Best regards,


More information about the Grok-dev mailing list