At Fri, 16 Aug 2002 09:30:44 +0100,
Andy Wardley wrote:
> 
> Looks good, with just a minor change of name.
> 
> I added it as html_line_break (rather than html_linebreak) and also 
> added html_para_break as an alias to the existing html_break, for the
> sake of completeness and consistency.

Perfect!

> > +=head2 html_linebreak
> > +
> > +This filter replaces any newlines with E<lt>brE<gt> HTML tags.
> 
> One other point: the documentation for the modules is built from the 
> docsrc tree (this is a separate distribution available from the website).
> It's easiest for me if you can edit the docsrc files direct (in this
> case, docsrc/lib/misc/filters) and send me patches against those.
> 
> Having said that, the whole docsrc tree is a mess and hard to figure out
> where everything is.  Changing the documentation directly in the module 
> file as you did is the second best way.  This is always preferable to not
> patching the documentation at all :-)

Ok, I'll follow this rule from the next patch of mine ;)

Thanks,

-- 
Tatsuhiko Miyagawa <[EMAIL PROTECTED]>

_______________________________________________
templates mailing list
[EMAIL PROTECTED]
http://www.template-toolkit.org/mailman/listinfo/templates

Reply via email to