[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:
http://core.trac.wordpress.org/browser/trunk/wp-includes/author-template.php
I would likely start at inserting around line 303 ...
Just a thought, didn't want to step on your review to suggest it directly.
Cais.
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
http://core.trac.wordpress.org/browser/trunk/wp-content/themes/twentyten/single.php,
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
http://tomlany.net
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>
http://lists.wordpress.org/mailman/listinfo/theme-reviewers
-------------- 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