Smartify - Recent Comments

Creating and modifying plugins.
Post Reply
Don Chambers
Regular
Posts: 3657
Joined: Mon Feb 13, 2006 2:40 am
Location: Chicago, IL, USA
Contact:

Smartify - Recent Comments

Post 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?????
=Don=
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Smartify - Recent Comments

Post 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
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Don Chambers
Regular
Posts: 3657
Joined: Mon Feb 13, 2006 2:40 am
Location: Chicago, IL, USA
Contact:

Post 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.
=Don=
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Post 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
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Don Chambers
Regular
Posts: 3657
Joined: Mon Feb 13, 2006 2:40 am
Location: Chicago, IL, USA
Contact:

Post by Don Chambers »

OK.... for now... :wink:
=Don=
Post Reply