[buddypress-trac] [BuddyPress Trac] #4677: Option to disable Group avatars

buddypress-trac noreply at wordpress.org
Fri Apr 17 10:09:26 UTC 2015

#4677: Option to disable Group avatars
 Reporter:  sooskriszta                           |       Owner:
     Type:  enhancement                           |      Status:  new
 Priority:  low                                   |   Milestone:  2.3
Component:  Component - Groups                    |     Version:
 Severity:  minor                                 |  Resolution:
 Keywords:  has-patch dev-feedback needs-testing  |

Comment (by hnla):

 Ok I've patched and tested this:

 * Style wise about the best that can be done generically for bp-default
 styles, adding the classes a good thing.

 * UX/UI doesn't feel right, if you're going to allow to disable avatars
 then you'll have to remove the group creation step

 * Similarly in terms of markup rendering  we have a mixed bag, group
 header if already had an avatar will still display the parent wrapper
 elements, group created after disablement won't, Group dir list displays
 the markup including empty anchor pointing to group if old or new group.

 We'll need to wrap all the avatar calls in a new check in the templates if
 we do this to remove all traces.

 We do impose a minor overhead albeit somewhat trivial of having to factor
 in additional styles possibly for companion sheets.

 If we do commit this to core what would be super nice is if we could have
 a brief write up or editing of existing guides for new UI functionality so
 that we can maintain up to date Codex references especially for the user
 guides section, with perhaps a @mention to myself or mercime so that we
 can record the addition and keep track.

Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4677#comment:26>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac

More information about the buddypress-trac mailing list