Changeset 36270

Timestamp:

01/12/2016 05:12:17 PM (8 years ago)

Author:

obenland

Message:

Themes: Clear floated theme cards on Themes page.

Also maintains visual separation for Broken Themes table on searches that
return no results.

See [36171] for .wp-clearfix.

Props 5um17, obenland, valendesigns, afercia.
Fixes #26646.

Location:

trunk/src/wp-admin

Files:

• css/themes.css (modified) (2 diffs)
• js/theme.js (modified) (2 diffs)
• theme-install.php (modified) (1 diff)
• themes.php (modified) (2 diffs)

trunk/src/wp-admin/css/themes.css

@@ -10 +10 @@
 .theme-browser .themes {
     clear: both;
-    padding: 0 0 100px;
 }
 
@@ -1125 +1124 @@
     font-style: normal;
     margin: 0;
-    padding: 0;
+    padding: 100px 0;
     text-align: center;
     display: none;
 }
+
 body.no-results p.no-themes {
     display: block;
 }
+
 body.show-upload-theme p.no-themes {
     display: none !important;

trunk/src/wp-admin/js/theme.js

@@ -81 +81 @@
         this.view.render();
         this.$el.empty().append( this.view.el ).addClass( 'rendered' );
-        this.$el.append( '<br class="clear"/>' );
     },
 
@@ -839 +838 @@
 themes.view.Themes = wp.Backbone.View.extend({
 
-    className: 'themes',
+    className: 'themes wp-clearfix',
     $overlay: $( 'div.theme-overlay' ),
 

trunk/src/wp-admin/theme-install.php

@@ -196 +196 @@
     <span class="spinner"></span>
 
-    <br class="clear" />
 <?php
 if ( $tab ) {

trunk/src/wp-admin/themes.php

@@ -230 +230 @@
 
 <div class="theme-browser">
-    <div class="themes">
+    <div class="themes wp-clearfix">
 
 <?php
@@ -283 +283 @@
 </div>
 <?php endforeach; ?>
-    <br class="clear" />
     </div>
 </div>