It was quite easy for me to slot in cached Libravatars, very few changes to make to imagecache_profiles.module or gravatar.module, and the improvement in appearance is fantastic.

Turns those grey empty faces into something less off-putting like this:
http://drupal.org/files/libravatars.png

Relies on this http://drupal.org/node/1300064#comment-5088008 to integrate Image Cache External
this http://drupal.org/node/334630#comment-5083794 to pass gravatars through to ICP
and whether or not we really need this #1302690: Theme file doing the same job as imagecache_profiles

CommentFileSizeAuthor
libravatars.png31.88 KBlightsurge

Comments

lightsurge’s picture

I also had to follow the advice here #1294104: Handle 404 and 403 errors. For some reason when new users sign up, some link in the chain breaks and imagcache_external attempts to fetch a bad url resulting in it throwing a wobbly of multiple drupal 404s, 403s, etc (probably one for each place the img would have been represented on the site).

So with so many changes as far as Commons is concerned I suppose it's more more likely that this will have to wait until Gravatar consumes Image Cache Profiles or vice versa. But it's one of those quick and apparently pointless things that users seem to like ;-)

lightsurge’s picture

In terms of the template file, it seems that all it does additionally is alter the img title text from "View user profile" to "[user_name]" for usability. It seems better to me to adjust imagecache_profiles to allow Views to override what it uses for the title text (by passing something through in &$variables), so I did that instead.

lightsurge’s picture

Issue summary: View changes

wrong link

japerry’s picture

Version: » 6.x-1.x-dev
Issue summary: View changes
Status: Active » Closed (won't fix)

Cleaning up the queue. No longer applicable in 7.x-3.x and we're commons 6.x-1.x is no longer supported.