[buddypress-trac] [BuddyPress Trac] #6570: Add UI for adding Profile Header Images for Users and Groups
buddypress-trac
noreply at wordpress.org
Fri Sep 4 22:45:36 UTC 2015
#6570: Add UI for adding Profile Header Images for Users and Groups
------------------------------------+------------------
Reporter: mercime | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 2.4
Component: API | Version:
Severity: normal | Resolution:
Keywords: dev-feedback has-patch |
------------------------------------+------------------
Comment (by imath):
As discussed during latest dev-chat i've been working on a new "user flow"
to set the user's avatar and cover image by adding 2 buttons directly
inside the header that are using an improved version of the patch i've
first posted on #6604.
Thanks to this modal box, i was able to also work on the wp-admin/extended
profile to set/delete/edit the user's cover image.
Then, @r-a-y was right gray header! So i have no more objections about it
Then, @r-a-y and @mercime were right about the options to disable/enable
the feature being a "regular" user, i've added them.
This made me realize that the `bp-templates/bp-legacy/buddypress
/buddypress-functions.php` is never loaded inside WP Admin as it's loaded
using `bp_locate_template()`. Meaning we can't use this file to set the
default feature of the template pack (so far only cover image). As a
result i've moved it inside `/bp-core/bp-core-theme-compatibility.php`.
Finally, i've added the cover image to the Groups component:
create/manage/administration screens. Here's a screenshot of it:
[[Image(https://cldup.com/bXWnO006qa.png)]]
So we have a first "full featured" patch. 6570.04.patch also includes
6604.patch so it's easier to test it. And precisely we need now to test it
to check everything is fine with all WordPress default themes. I've added
filters and actions, but feel free to suggest new ones at key points.
Also, once this gets in, maybe we should think about a new activity action
e.g.: "imath changed their cover image"
Oh and one more thing :) Here's a video demonstrating the patch
https://vimeo.com/138356216
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6570#comment:38>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list