#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