Re: First round of comments merging done

Steven Armstrong <[email protected]>
Newsgroups gmane.comp.web.pyblosxom.devel
Message-ID <[email protected]>
will guaraldi wrote:
> Uh, that's one change that I would really like to be made.  There are a 
> bunch of plugins that get totally hosed because of this.  To be honest, 
> I really think it's a bad idea to stomp on the existing template with 
> the comment-story anyhow and there's no good reason that I can think of 
> to do so.
> 
> Anyhow, if you don't make the change, I'll have to fork the comments 
> plugin and offer my own version that works with these other plugins.
> 
> /will
> 
> 

After thinking a bit about the 'template stomping issue' I think Will is 
right. When I look at my comment-story.html, its pretty much the same as 
story.html anyway. I'll rewrite my html and css to work 'the Will way'. 
Makes sense to me not to have redundant html snippets arround.

I think before forking the plugin it would be a good idea to talk about 
it and make a feature list or so.

Something like:
what's allready coded
what's good for everyone
what can be made configurable
what's wantend, but not yet coded
...

then refactor the whole thing. Maybe make a comment-core plugin which 
can be used/extended by others?

any comments on this comment? ;)

cheers
Steven


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now. 
http://productguide.itmanagersjournal.com/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.