Page 1 of 1

Smartify - Recent Comments

Posted: Thu Dec 13, 2007 3:20 pm
by Don Chambers
Just to keep you on your toes Garvin - how about smarty for the recent comments plugin??? That one would be really nice! Especially when used outside the sidebar. I, for one, would like to make recent comments look nearly identical to regular comments.... maybe it could even use comments.tpl?????

Re: Smartify - Recent Comments

Posted: Thu Dec 13, 2007 3:44 pm
by garvinhicking
Hi!
Don Chambers wrote:Just to keep you on your toes Garvin - how about smarty for the recent comments plugin??? That one would be really nice! Especially when used outside the sidebar. I, for one, would like to make recent comments look nearly identical to regular comments.... maybe it could even use comments.tpl?????
I don't see the need for that, unlike to the rss sidebar plugin. Why not use the smarty {fetchcomments} call and use comments.tpl to format things outside the sidebar?

Regards,
Garvin

Posted: Thu Dec 13, 2007 3:48 pm
by Don Chambers
That is probably a good suggestion for outside the sidebar, but I have also thought the format within the sidebar could use a little better presentation than what can currently be accomplished just with css. Considering how popular this sidebar plugin is, I just thought enabling smarty would be a logical step.

Posted: Thu Dec 13, 2007 4:05 pm
by garvinhicking
Hi!

IMHO the performance impact should really be kept low. Especially for such a popular plugin.

Despite the RSS sidebar plugin, we have no other item fields that can be displayed there, we already show everything. IMHO basically anything should be achievable with CSS, and if not, we should instead add the right CSS classes to the plugin to enhance it.

The less smarty templates need to be parsed inthe sidebars, the more power you have in performance. Smarty really hurts for small singular .tpl files.

Regards,
Garvin

Posted: Thu Dec 13, 2007 5:46 pm
by Don Chambers
OK.... for now... :wink: