Skip to:
Content

Opened 17 months ago

Last modified 8 months ago

#4677 new enhancement

Option to disable Group avatars

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

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 11 months ago.

Download all attachments as: .zip

Change History (10)

comment:1 boonebgorges17 months 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 sooskriszta17 months 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 Fire17 months 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 sooskriszta17 months ago

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

comment:5 DJPaul17 months ago

  • Milestone changed from 1.6.2 to 1.7

comment:6 johnjamesjacoby17 months ago

  • Milestone changed from 1.7 to 1.8

No patch. Punting to 1.8.

comment:7 boonebgorges11 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?

boonebgorges11 months ago

comment:8 boonebgorges11 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 sooskriszta8 months ago

This ought to do it.

Note: See TracTickets for help on using tickets.