Re: Small patch for loop macro

Ok, I just committed my patch. If anybody thinks this was premature
(e.g. because we should discuss the itemPrefix/Suffix parameter names,
or implementation) please drop a message.

hannes


2007/5/10, Hannes Wallnoefer <[email protected]>:
> Hi Maks,
>
> thanks for the patch. The itemPrefix/Suffix parameters make sense, but
> by composing a new string for each item instead of just using
> res.write() you add a lot of overhead. I slightly modified your patch,
> see attachment. Can you (or somebody else) please give it a try?
>
> hannes
>
> 2007/5/10, Maksim Lin for technical support mailling lists
> <[email protected]>:
> > Hi,
> >
> > Ever since I discovered the loop macro (via the new reference docs) I've
> > found it *very* handy but one thing I missed was the ability to set a
> > prefix/suffix on each individual items skin render. So I've attached a
> > small patch to add this functionality to HopObject loop macro.
> >
> > I find it very handy since sometimes I'm rendering items of a collection
> > as HTML list items, sometimes as options in a select element, sometimes
> > as just a comma seperated list and this way I can do it all with one
> > very simple skin.
> >
> > Maks.
> >
> > _______________________________________________
> > Helma-user mailing list
> > [email protected]
> > http://helma.org/mailman/listinfo/helma-user
> >
> >
> >
>
>