Changeset 22806

Timestamp:

11/22/2012 03:19:08 AM (14 years ago)

Author:

koopersmith

Message:

Media: Check to see if we need more attachments when the browser is attached to the DOM. fixes #22480, see #21390.

File:

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

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

@@ -2573 +2573 @@
                 },
 
-                destroy: function() {
+                dispose: function() {
                         this.collection.props.off( null, null, this );
-                        this.collection.off( 'add remove reset', null, this );
-                        this.model.off( 'change:edge change:gutter', this.css, this );
                         $(window).off( 'resize.attachments', this._resizeCss );
-                        this.remove();
+                        media.View.prototype.dispose.apply( this, arguments );
                 },
 
@@ -2681 +2679 @@
                         }, this ) );
 
-                        // Then, trigger the scroll event to check if we're within the
+                        return this;
+                },
+
+                ready: function() {
+                        // Trigger the scroll event to check if we're within the
                         // threshold to query for additional attachments.
                         this.scroll();
-
-                        return this;
                 },