double_d
The code doesn't really follow the MVC pattern but this is not the point.
Soorry but I find really silly that the javascript is loaded everytime, even if in the sections in which commenting is disabled.
I find strange then that the comments are injected in the page with javascript rather than being a part of the xhtml.
What about accessibility?
Last thing, i find the publish/unpublish mecchanism from frontend really tricky when i tried to change its look.
I think instead of planning to add new functionalities (useful for who?) you should concentrate in improving the code, performance and making accessible what you already have done.
PS: regarding accessibility have a think about an audio captcha.
1. This component was developed and at this supports both Joomla 1.0 and Joomla 1.5. Because Joomla 1.0 didn't support MVC I have use my own implementation of templates... However in current implementation the templates they are fully separated from code. And you also can overload JComments template in your Joomla's template. In nearest time I'm going to stop supporting Joomla 1.0 and begin using Joomla 1.5 MVC implementation. I think it would be more correct and more productively.
2. The comments are injected in the page with JavaScript only if caching activated in Joomla. This is anti-cache trick but it could be disabled.
3. I don't understand what problems have you got with changing look of these buttons.
4. If you've any suggestions about improvement code and performance - please go to our forum and post them in appropriate forum section.
Thank you for your criticism, but do not forget that is always easier to criticize than to do something ;) Visit us on the forum and let's discuss how to make this component better.
I had a problem in a first moment because i didn't understand that there shouldn't be any space between the separated JS and some JS for IE6 only didn't get excluded.
I have a little issue still, i would like to compress also the css for print, but since it has different styles than the screen css they can't be toghether in one file, so i think an option that let compress also the some excluded css a part would be the GREAT.
I designed a template for a client that in the end of each article has some links (like feed, register, twitter) and then some nice icons to share.
I used this without external css and adapting it to the style i prepared and it works nicely.
Thanks
