Ticket #25962: 25962.diff

wp-admin/js/theme.js

@@ -61 +61 @@
                 var view,
                         self = this;
 
+                // Don't render the search if the user cannot see all the themes
+                if ( ! themes.data.settings.switchThemes ) {
+                        return;
+                }
+
                 view = new themes.view.Search({ collection: self.collection });
 
                 // Render and append after screen title
@@ -163 +168 @@
                 collection = _( collection.first( 15 ) );
 
                 return collection;
+        },
+
+        singleThemeMode: function() {
+                var current = this.findWhere({ active: true });
+
+                if ( current ) {
+                        this.reset( current );
+                }
         }
 });
 
@@ -408 +421 @@
                 // Clear the DOM, please
                 this.$el.html( '' );
 
+                // If the user doesn't have switch capabilities
+                // render the detailed view of the active theme
+                if ( ! themes.data.settings.switchThemes ) {
+                        // Sets up the collection changes
+                        this.collection.singleThemeMode();
+
+                        // Constructs the view
+                        this.singleTheme = new themes.view.Details({
+                                model: this.collection.models[0]
+                        });
+
+                        // Render and apply a 'single-theme' class to our container
+                        this.singleTheme.render();
+                        this.$el.addClass( 'single-theme' );
+                        this.$el.append( this.singleTheme.el );
+                }
+
                 // Generate the themes
                 // Using page instance
                 this.renderThemes( this.parent.page );

wp-admin/css/wp-admin.css

@@ -7052 +7052 @@
 .theme-overlay .parent-theme strong {
         font-weight: 700;
 }
+/**
+ * Single Theme Mode
+ * Displays detailed view inline when a user has no switch capabilities
+ */
+.single-theme .theme-overlay .theme-backdrop,
+.single-theme .theme-overlay .theme-header,
+.single-theme .theme {
+        display: none;
+}
+.single-theme .theme-overlay .theme-wrap {
+        clear: both;
+        position: static;
+}
+.single-theme .theme-overlay .theme-about {
+        padding: 30px;
+        position: static;
+}
+.single-theme .theme-overlay .theme-actions {
+        position: static;
+}
 
 /**
  * Basic Responsive structure...

wp-admin/themes.php

@@ -100 +100 @@
                 'confirmDelete' => __( "Are you sure you want to delete this theme?\n\nClick 'Cancel' to go back, 'OK' to confirm the delete." ),
                 'root'          => admin_url( 'themes.php' ),
                 'extraRoutes'   => '',
+                'switchThemes'  => false, // needs proper logic here
         ),
         'l10n' => array(
                 'addNew' => __( 'Add New Theme' ),