Changeset 22690

Timestamp:

11/19/2012 10:48:11 PM (12 years ago)

Author:

koopersmith

Message:

Media: Add render tracking to views. see #21390.

File:

• trunk/wp-includes/js/media-views.js (modified) (6 diffs)

trunk/wp-includes/js/media-views.js

@@ -683 +683 @@
             }, this );
 
-            if ( ! options.silent )
+            if ( ! options.silent ) {
+                _.each( views, this._maybeRender, this );
                 this[ method ]( $selector, els, options );
+            }
 
             return this;
@@ -756 +758 @@
             var root = this._views[''];
 
+            _.each( this.all(), this._maybeRender, this );
+
             if ( root )
                 this.replace( this.view.$el, _.pluck( root, 'el' ) );
@@ -764 +768 @@
             }, this );
 
+            this.rendered = true;
             return this;
         },
@@ -814 +819 @@
 
             return this;
+        },
+
+
+        // #### Internal. Maybe render a view.
+        _maybeRender: function( view ) {
+            if ( ! view.views || view.views.rendered )
+                return;
+
+            view.render();
+            view.views.rendered = true;
         }
     });
@@ -1149 +1164 @@
 
                 AttachmentView: state.get('AttachmentView')
-            }).render() );
+            }) );
         },
 
@@ -1156 +1171 @@
             this.content.view( new media.view.UploaderInline({
                 controller: this
-            }).render() );
+            }) );
         },