All the avatars are different sizes how do i scale it so all of them stay one size?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

stella’s picture

Category: support » feature
Status: Active » Closed (won't fix)

There's no support for imagecache at present, and I don't really plan on adding this feature unless more people request it, sorry.

Cheers,
Stella

shrike202’s picture

consider this a request, it would help streamline

pribeh’s picture

request.

seehawk’s picture

I'd love to see imagecache integrated into this module, so I'm adding my voice to the list of requesters.

In fact, there's a module called ImageCache Profiles that already works really nicely, and if this module could work in tandem with it, it would make this module really powerful.

ttnk’s picture

request too please :D
thanks

colemanw’s picture

I'd like to clarify that imagecache works great with this module, in terms of scaling the user's profile picture once they have chosen one (in comments, posts, their profile page, etc). However, allowing it to work its magic on the selection list would be a great additional feature. So add my name to the list of requests, and thanks for your great work on this module!

PixelClever’s picture

Title: How do i scale all avatars through imagecache? » Add theme function for avatar selection (so that imagecache can be used)
Status: Closed (won't fix) » Active

I can understand you not wanting to add full imagecache integration. However, it would be nice to at least have a theme function which allows the images to be resized and formated as needed.

mherchel’s picture

Request!

chrisdawson’s picture

Request :)

chrisdawson’s picture

Version: 6.x-1.5 » 6.x-1.x-dev
Status: Active » Needs review
FileSize
2.78 KB

Great module Stella! I wanted imagecache support for my own project, so I've added it :)

Here's my patch - please could you test and review. This is my first time submitting a patch to Drupal so please let me know if I've done anything incorrectly.

Thanks

skizzo’s picture

I tried the patch and it works fine, thank you. One thing I noticed is that the image is missing the width and height attributes, affecting page speed. Should that be reported as a separate issue?

stella’s picture

Status: Needs review » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

bcmiller0’s picture

that patch is leaving a *.orig file when applied. Trying a git patch version to see if differs when used with our apci.make.

stella’s picture

or you can just use the dev version since it was committed already