[theme-reviewers] Fwd: Re: Distilled Review

Tom Lany mail at tomlany.net
Thu Jun 17 15:03:09 UTC 2010

Thought I would pass along this idea.

-------- Original Message --------
Subject: 	Re: [theme-reviewers] Distilled Review
Date: 	Thu, 17 Jun 2010 08:16:16 -0400
From: 	Edward Caissie <edward.caissie at gmail.com>
To: 	mail at tomlany.net

Tom -

You might suggest to "randall" to copy and re-write the 
'wp_list_authors" function and include an option for the use of Gravatars.

Based on the code in trunk here: 

I would likely start at inserting around line 303 ...

Just a thought, didn't want to step on your review to suggest it directly.


On Wed, Jun 16, 2010 at 4:38 PM, Tom Lany <mail at tomlany.net 
<mailto:mail at tomlany.net>> wrote:

    Have you tried <?php echo get_avatar( get_the_author_meta(
    'user_email' )); ?> , or something similar for pulling the Avatar? 
    Look on line 37 of
    as I suggested below for the code used in the default  (in 3.0)
    WordPress theme.  They add a gravatar to single.php without a DB query.

    Tom Lany

    On 6/16/10 1:42 AM, randall wrote:
>     On 06/15/2010 11:37 PM, Tom Lany wrote:
>>     Hi Randall,
>>     Thanks for taking a look at some of these issues.
>>     Regarding the DB query, you should be able to pull an avatar
>>     without a direct database query.  See line 37 of:
>>     http://core.trac.wordpress.org/browser/trunk/wp-content/themes/twentyten/single.php
>>     for an example of how this is done in Twentyten (the new default
>>     theme).
>>     Regarding the "image is displayed twice" issue, when I tested the
>>     theme, I saw two copies of the same image on an attachment image
>>     page.  To reproduce, upload an image on a post page.  For the
>>     link option in the image option dialog that appears once the
>>     image has been uploaded, choose "Post URL", and then insert the
>>     image into the post.  Click on the link, and you should see the
>>     double inserted image.
>>     Thanks!
>>     Tom Lany
>>     http://tomlany.net
>     thanks Tom,
>     have added an attachment.php and this seems to fix the issue of
>     the double images.
>     leaves the db query, pulling an avatar like on line 37 of the link
>     is what i have in my single.php as well, this pulls the data +
>     avatar of the post author,
>     but, what i cant figure out is to have a list of all the blogs
>     authors with there avatars, when hitting the search engines for
>     this it seems to confirm that it is not possible without a direct
>     query as described here
>     http://codex.wordpress.org/Function_Reference/wpdb_Class.
>     but maybe i'm overlooking something here....would love to be
>     proven wrong.
>     in case anybody knows some objection regards to safety/performance
>     etc.. , it would be simple to remove the query and have it
>     replaced with a list without any avatars, most bloggers are ugly
>     anyway ;)
>     _______________________________________________
>     theme-reviewers mailing list
>     theme-reviewers at lists.wordpress.org
>     <mailto:theme-reviewers at lists.wordpress.org>
>     http://lists.wordpress.org/mailman/listinfo/theme-reviewers

    theme-reviewers mailing list
    theme-reviewers at lists.wordpress.org
    <mailto:theme-reviewers at lists.wordpress.org>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/mailman/private/theme-reviewers/attachments/20100617/1c28ab48/attachment.htm>

More information about the theme-reviewers mailing list