Changeset 12447

Timestamp:

08/16/2019 01:50:51 PM (7 years ago)

Author:

dcavins

Message:

Nouveau: Change AJAX template loading event emission timing.

In Nouveau, a useful event, bp_ajax_request, is emitted when a template
is updated wth new content. However, the new content is eased in using
a fadeIn transition. This changeset moves the event to the fadeIn complete
so that any actions tied to bp_ajax_request will be operating on
visible and full-size elements.

Fixes #8128.

File:

• trunk/src/bp-templates/bp-nouveau/js/buddypress-nouveau.js (modified) (1 diff)

trunk/src/bp-templates/bp-nouveau/js/buddypress-nouveau.js

@@ -319 +319 @@
                                                         $( data.target ).fadeOut( 100, function() {
                                                                 self.inject( this, response.data.contents, data.method );
-                                                                $( this ).fadeIn( 100 );
-
+                                                                $( this ).fadeIn( 100, 'swing', function(){
+                                                                        // Inform other scripts the list of objects has been refreshed.
+                                                                        $( data.target ).trigger( 'bp_ajax_request', $.extend( data, { response: response.data } ) );
+                                                                } );
+                                                        } );
+                                                } );
+
+                                        } else {
+                                                $( data.target ).fadeOut( 100, function() {
+                                                        self.inject( this, response.data.contents, data.method );
+                                                        $( this ).fadeIn( 100, 'swing', function(){
                                                                 // Inform other scripts the list of objects has been refreshed.
                                                                 $( data.target ).trigger( 'bp_ajax_request', $.extend( data, { response: response.data } ) );
                                                         } );
-                                                } );
-
-                                        } else {
-                                                $( data.target ).fadeOut( 100, function() {
-                                                        self.inject( this, response.data.contents, data.method );
-                                                        $( this ).fadeIn( 100 );
-
-                                                        // Inform other scripts the list of objects has been refreshed.
-                                                        $( data.target ).trigger( 'bp_ajax_request', $.extend( data, { response: response.data } ) );
                                                 } );
                                         }