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

[buddypress-trac]

#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 boonebgorges):

 imath - this is all great. Thanks so much. I had forgotten/neglected most
 of these points.

 hnla - imath's patch addresses your point about group creation.

 I intentionally tried not to touch the templates. But I agree that it's
 crummy to leave empty markup in there. And now that I'm doing the trick
 with the new 'group-has-avatar' class, removing the markup itself will be
 a sort of progressive enhancement, which means it's not such a big deal to
 mod the templates. [attachment:4677.5.patch] does what are (I think) all
 the necessary template checks, and it looks to me like the page flow
 continues to be fine in these cases.

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