Skip to:
Content

Opened 2 years ago

Last modified 9 months ago

#4677 new enhancement

Option to disable Group avatars

Reported by: sooskriszta Owned by:
Milestone: Future Release Priority: low
Severity: minor Version:
Component: Component - Groups Keywords: has-patch dev-feedback
Cc: vivek@…

Description

In Admin, it would be nice to have an option to disable avatars for Groups.

I'm assigning component Groups as opposed to Theme, as it is not just a question of showing the avatar. It is a wholesale question of uploading too - so group creation workflow, group edit options, etc.

Attachments (1)

4677.patch (5.5 KB) - added by boonebgorges 22 months ago.

Download all attachments as: .zip

Change History (13)

comment:1 @boonebgorges2 years ago

  • Milestone changed from Awaiting Review to 1.6.2
  • Priority changed from normal to low
  • Severity changed from normal to minor

This may not warrant a UI toggle, but it's a good idea to make it possible with a constant or a filter.

comment:2 @sooskriszta2 years ago

As a very non-tech-literate user, I always advocate for a bigger admin UI footprint for BuddyPress. The more easily folks can find and use features, the better it is...

comment:3 @Prometheus Fire2 years ago

I agree that a UI toggle seems like a lot of extra work for this. The use cases for this need are limited and not worth having a UI switch, but a filter or define statement in wp-config.php would be great way to turn this off.

Not all BP instances (corporate intranet, for ex.) would need avatars for groups and being able to disable would be a good flexibility.

comment:4 @sooskriszta2 years ago

A wp-config.php statement would be relatively more usable than just a filter.

comment:5 @DJPaul2 years ago

  • Milestone changed from 1.6.2 to 1.7

comment:6 @johnjamesjacoby2 years ago

  • Milestone changed from 1.7 to 1.8

No patch. Punting to 1.8.

comment:7 @boonebgorges22 months ago

  • Keywords has-patch dev-feedback added

4677.patch is a bare-bones pass at implementing a toggle. To toggle, put the following in bp-custom.php:

add_filter( 'bp_enable_group_avatars', '__return_false' );

As you see, it requires some template-level changes. Not ideal, but it's the only way to do it.

Styling probably also needs some tweaks when avatars are absent, especially in the groups directory. Could put in a no-avatar class or something along those lines, or could try reworking the way that the markup and styling works so that the spacing naturally readjusts.

Anyone have thoughts?

@boonebgorges22 months ago

comment:8 @boonebgorges21 months ago

  • Milestone changed from 1.8 to Future Release

Punting this one. If anyone has feedback or suggestions, we can do it for 1.9.

comment:9 @sooskriszta19 months ago

This ought to do it.

comment:10 @sooskriszta11 months ago

  • Cc vivek@… added

comment:11 @sooskriszta9 months ago

  • Cc vivek@… removed

comment:12 @sooskriszta9 months ago

  • Cc vivek@… added
Note: See TracTickets for help on using tickets.