Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 10 years ago

Last modified 8 years ago

#6372 closed enhancement (fixed)

Use "Mystery Group" avatar when group avatar not available.

Reported by: sooskriszta Owned by: boonebgorges
Priority: normal Milestone: 2.6
Component: Groups Version:
Severity: normal Keywords: has-patch commit
Cc: vivek@…

Description

At the moment, if a group avatar has not been uploaded then BuddyPress shows the "mystery man" avatar instead (just as it does for user profile photos)

Instead, we should use a "mystery group" avatar, to distinguish between groups and users. Something like this (but obviously, more in tune with the mystery man style)

Attachments (4)

3196412953-074af62ea5ff218b6a6eeab89104f616.png​ (3.2 KB ) - added by sooskriszta 12 years ago.
Mystery man
group.svg​ (1.1 KB ) - added by sooskriszta 12 years ago.
Mystery group
groups-default-avatar#10.zip​ (8.7 KB ) - added by abweb 11 years ago.
Archive contains patch file and new image for groups avatar
6372.diff​ (6.8 KB ) - added by boonebgorges 11 years ago.

Download all attachments as: .zip

Change History (18)

@sooskriszta
12 years ago

Mystery group

#1 @sooskriszta
12 years ago

  • Cc vivek@… added

#2 @hnla
12 years ago

Like the notion!

#3 @DJPaul
11 years ago

  • Keywords has-patch early added
  • Milestone Awaiting Review → Future Release

Ooh, interesting.

cc @imath

#4 follow-up: @imath
11 years ago

Thanks for the ping DJPaul, this is interesting! We could actually use the BuddyPress logo :)

#5 in reply to: ↑ 4 @sooskriszta
11 years ago

Replying to imath:

We could actually use the BuddyPress logo

Great idea

#6 @sooskriszta
11 years ago

  • Summary User "Mystery Group" avatar when group avatar not available. → Use "Mystery Group" avatar when group avatar not available.

#7 follow-up: @DJPaul
11 years ago

  • Keywords needs-patch added; has-patch early removed

"Mystery Person", please. :) Let's

#8 @netweb
11 years ago

r1004 missed a ticket reference, here it is:

Changeset r10041
Mystery person, not mystery man.
More to do here, but it's a start.

#9 in reply to: ↑ 7 @sooskriszta
11 years ago

Replying to DJPaul:

"Mystery Person", please.

Mystery Man ​https://codex.wordpress.org/Function_Reference/get_avatar ;-)

@abweb
11 years ago

Archive contains patch file and new image for groups avatar

@boonebgorges
11 years ago

#10 @boonebgorges
11 years ago

  • Keywords has-patch 2nd-opinion added; needs-patch removed
  • Milestone Future Release → 2.6
  • Type defect (bug) → enhancement

Image looks great.

6372.diff​ makes the following improvements:

  • I changed the implementation a little bit so that the default kicks in before the 'bp_core_default_avatar_group' filter. This ensures that existing filters here will continue to work as expected.
  • Added a 50x50 image, and support for loading the thumbnail version when necessary.

Question for the team: Why do we ever ping Gravatar for group images? They will never match. I guess if you're getting a Wavatar or some other generated image, maybe you want the variety? In 6372.diff​ I completely disabled Gravatar for group images, but maybe this is too extreme. Maybe we should only disable if avatar_default is set to mm? (See #4571 and [7769].)

This ticket was mentioned in ​Slack in #buddypress by dcavins. ​View the logs.


11 years ago

#12 @DJPaul
10 years ago

  • Keywords commit added

@boonebgorges I think that was done because we thought it might be useful to site owners who register matching email addresses with Gravatar. It sounds pretty ridiculous now. I don't think we ever gave the impact of additional requests any consideration at the time.

#13 @boonebgorges
10 years ago

  • Keywords 2nd-opinion removed

Thanks, @DJPaul. I am going to go ahead and disable Gravatar requests for groups across the board. This is going to result in a change in behavior for people who are expecting Wavatar etc for their groups, but the value can be filtered for anyone who cares. Let's see if we annoy anyone :)

#14 @boonebgorges
10 years ago

  • Owner set to boonebgorges
  • Resolution → fixed
  • Status new → closed

In 10721:

Introduce 'mystery group' avatar for groups without custom avatar.

This nifty new image is totally grouperiffic, and it's gonna supercharge
your BuddyPress community, to the extreme.

As part of the improvement, Gravatar requests for group avatars have been
disabled by default. Gravatar requests never match for groups, anyway, so
this should save HTTP requests in most cases. Filter 'bp_core_fetch_avatar_no_grav'
to restore the previous behavior.

Props sooskriszta, abweb.
Fixes #6372.

Note: See TracTickets for help on using tickets.