Changeset 12447

Timestamp:

08/16/2019 01:50:51 PM (6 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 } ) );
                         } );
                     }