Changeset 13306 for trunk/src/bp-templates/bp-nouveau/buddypress-functions.php

Timestamp:

07/26/2022 02:59:36 PM (2 years ago)

Author:

imath

Message:

First step towards only loading Template Packs assets in BP pages

Avoid loading these assets everywhere inside the blog is a frequent request made by BuddyPress users. In 11.0.0 we are doing a first step into this direction by making available a new filter to restrict Template Packs assets loading to BuddyPress pages.

We encourage these users to simply add the add_filter( 'bp_enqueue_assets_in_bp_pages_only', '__return_true' ); code into a BP Custom file for instance to contribute to testing whether it has an unwanted side effect on the BuddyPress plugins that they might have activated.

This will help us feel more secure about completely restricting these assets to BP pages in a second step.

Props dcavins

Fixes #8679

File:

• trunk/src/bp-templates/bp-nouveau/buddypress-functions.php (modified) (2 diffs)

trunk/src/bp-templates/bp-nouveau/buddypress-functions.php

@@ -179 +179 @@
         // Scripts & Styles.
         $registration_params = array(
-            'hook'     => 'bp_enqueue_scripts',
+            'hook'     => 'bp_enqueue_community_scripts',
             'priority' => 2,
         );
@@ -196 +196 @@
 
         // Enqueue theme CSS.
-        add_action( 'bp_enqueue_scripts', array( $this, 'enqueue_styles' ) );
+        add_action( 'bp_enqueue_community_scripts', array( $this, 'enqueue_styles' ) );
 
         // Enqueue theme JS.
-        add_action( 'bp_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
+        add_action( 'bp_enqueue_community_scripts', array( $this, 'enqueue_scripts' ) );
 
         // Enqueue theme script localization.
-        add_filter( 'bp_enqueue_scripts', array( $this, 'localize_scripts' ) );
-        remove_action( 'bp_enqueue_scripts', 'bp_core_confirmation_js' );
+        add_action( 'bp_enqueue_community_scripts', array( $this, 'localize_scripts' ) );
+        remove_action( 'bp_enqueue_community_scripts', 'bp_core_confirmation_js' );
 
         // Body no-js class.