Not sure if this is image cache or a theme setting. But id like to display the user's name when hovering over their imagecache thumbnail. Right now, it says "view user profile" for any given user.

Thanks
Chris

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

glitz’s picture

Project: ImageCache » ImageCache Profiles
Version: 6.x-1.0-alpha2 » 6.x-1.3
parashutiki’s picture

subscribe

parashutiki’s picture

Hack =)

          $attributes = array('attributes' => array('title' => $alt), 'html' => TRUE);
          $image = theme('imagecache', $preset['presetname'], $picture, $alt, $alt);
mattiasj’s picture

Could this be done in a nice way even though we cant override preprocess_user_picture?

andypost’s picture

any ideas?

dawehner’s picture

Status: Active » Needs review
FileSize
745 bytes

What about doing the same thing as the user.module.

Here is a patch for it.

andypost’s picture

Version: 6.x-1.3 » 6.x-1.x-dev

Looks reasonable as template_preprocess_user_picture() core's implementation

can anyone test this patch?

vinoth.3v’s picture


if (!empty($account->uid) && user_access('access user profiles')) {
  $title = t('view !name\'s profile', array('!name' => $account->name));

dawehner’s picture

Status: Needs review » Needs work
+++ b/imagecache_profiles.moduleundefined
@@ -52,7 +52,7 @@ function imagecache_profiles_preprocess_user_picture(&$variables) {
           $title = t('View user profile.');

Do we really need this variable here?

andypost’s picture

+++ b/imagecache_profiles.moduleundefined
@@ -52,7 +52,7 @@ function imagecache_profiles_preprocess_user_picture(&$variables) {
           $attributes = array('attributes' => array('title' => $title), 'html' => TRUE);

Used in title for label

Powered by Dreditor.

andypost’s picture

Status: Needs work » Needs review
FileSize
916 bytes

I think we should follow Core's implementation as I pointed in #7

dawehner’s picture

From my perspective this path is RTBC, but i guess because i'm the original author of the patch i can't mark it for myself.

andypost’s picture

@dereine I'm ok to commit this patch but can't test, the question mostly in code style :)

andypost’s picture

Status: Needs review » Fixed

Thanx, I've commited #11

Status: Fixed » Closed (fixed)

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