Ticket #36613: 36613.4.diff

src/wp-admin/js/theme.js

@@ -491 +491 @@
                 // Set focus to current theme.
                 themes.focusedTheme = this.$el;
 
-                // Construct a new Preview view.
-                themes.currentPreview = preview = new themes.view.Preview({
+
+                themes.preview = preview = new themes.view.Preview({
                         model: this.model
                 });
 
@@ -573 +573 @@
                         self.current = self.model;
                 });
 
-                // Listen for closepreview events, closing the preview.
-                this.listenTo( preview, 'closepreview', function() {
-                        preview.close();
-                });
         },
 
         // Handles .disabled classes for previous/next buttons in theme installer preview
@@ -907 +903 @@
         },
 
         close: function() {
+
                 this.$el.fadeOut( 200, function() {
                         $( 'body' ).removeClass( 'theme-installer-active full-overlay-active' );
 
@@ -1914 +1911 @@
                 // Queries the API for the passed theme slug
                 themes.router.on( 'route:preview', function( slug ) {
 
+                        // Construct a new Preview view.
+                        if ( themes.preview ) {
+                                themes.preview.undelegateEvents();
+                                themes.preview.unbind();
+                        }
+
                         // If the theme preview is active, set the current theme.
                         if ( self.view.view.theme && self.view.view.theme.preview ) {
                                 self.view.view.theme.model = self.view.collection.findWhere( { 'slug': slug } );
@@ -1943 +1946 @@
                         self.view.sort( sort );
 
                         // Close the preview if open.
-                        if ( themes.currentPreview ) {
-                                themes.currentPreview.trigger( 'closepreview' );
+                        if ( themes.preview ) {
+                                themes.preview.close();
                         }
                 });