Changeset 22806

Timestamp:

11/22/2012 03:19:08 AM (13 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;
         },