[buddypress-trac] [BuddyPress Trac] #6772: BuddyPress Embeds for activity, user profiles, groups

buddypress-trac noreply at wordpress.org
Thu May 5 22:58:28 UTC 2016


#6772: BuddyPress Embeds for activity, user profiles, groups
------------------------------------+------------------
 Reporter:  imath                   |       Owner:
     Type:  idea                    |      Status:  new
 Priority:  normal                  |   Milestone:  2.6
Component:  API                     |     Version:
 Severity:  normal                  |  Resolution:
 Keywords:  dev-feedback has-patch  |
------------------------------------+------------------

Comment (by imath):

 Hi @r-a-y

 First i'd like to say you did a really great work on this ticket. I've
 really tried to convert myself about "embed in embed", to prove it here's
 the beginning of a suggestion patch i was about to make ->
 [https://gist.github.com/imath/43105d0241c330f6a12ba01044b672b7 gist]

 But after embedding an activity containing a youtube video or a vimeo
 video into a post, this is what i got:
 https://cloudup.com/cXrNHE63sjx

 The 2 videos refused to display or play because of a cross domain issue.
 So i definitively think we shouldn't try to iframe an iframe! Because
 anyway we can't see the content.

 Then, attaching the inline embed css to open-sans might not be a good
 idea, because for some reason, WordPress 4.6 is not including the css
 anymore, meaning the embed css is not attached :(

 Then, i think we should use the Activity action as the title of the embed,
 as it gives a nice piece of information when for instance an activity is
 posted within a group.

 Finally i understand the will to display some content for the embed so i'm
 suggesting to fetch the image of it so that's it's better than nothing ;)
 And image added to activity would also be represented using this
 suggestion...

 So after applying your patch, i've quickly drafted what i have in mind an
 put it into 6772.07.suggestions.patch (attached to this ticket)
 NB: you'll first need to apply 6772.07.patch

 And here's some before after:
 [[Image(https://cldup.com/DgLvCAsS_m.jpg)]]
 Youtube

 [[Image(https://cldup.com/CKXME_6bD8.jpg)]]
 Vimeo

 [[Image(https://cldup.com/j3d09LilaJ.jpg)]]
 A Group activity containing an image

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


More information about the buddypress-trac mailing list