[theme-reviewers] Fwd: Re: Distilled Review

[randall]

you guys were right, i was wrong.

have rewritten authors-page.php

randall


get_users_of_blogOn 06/17/2010 05:03 PM, Tom Lany wrote:
> 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
>
>
>


-- 
web design&  hosting
http://www.ano-hosting.com

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