[buddypress-trac] [BuddyPress Trac] #8340: Support native browser image lazy-loading

buddypress-trac noreply at wordpress.org
Thu Sep 24 21:50:13 UTC 2020

#8340: Support native browser image lazy-loading
 Reporter:  r-a-y        |       Owner:  r-a-y
     Type:  enhancement  |      Status:  assigned
 Priority:  normal       |   Milestone:  7.0.0
Component:  Core         |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |

Comment (by r-a-y):

 >  It works as expected in many places except when a post has been embed
 into an activity in BP Nouveau. Maybe it's best to only add lazy loading
 to images ?

 Hi imath, can you tell me what you mean here? If I post an activity update
 from the Sitewide Activity page with a YouTube link, the `<iframe>`
 includes the `loading="lazy"` attribute with the patch applied. I've
 attached a unit test. Be sure to uncomment the line in the patch to test
 the addition of the `loading` attribute.

 I noticed that oEmbed isn't working on activity replies after the initial
 AJAX response in Nouveau. It works after the page is loaded. However, it
 works in bp-legacy as expected. Is that what you mean? If so, I'll work on
 fixing that.

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

More information about the buddypress-trac mailing list