Changeset 20455 for trunk/wp-admin/includes/class-wp-themes-list-table.php

Timestamp:

04/12/2012 09:10:40 PM (13 years ago)

Author:

koopersmith

Message:

Theme list table style improvements. Make 'delete' link red and separate from other actions. New action separators. see #20403.

File:

• trunk/wp-admin/includes/class-wp-themes-list-table.php (modified) (3 diffs)

trunk/wp-admin/includes/class-wp-themes-list-table.php

@@ -142 +142 @@
                 . '<a href="#" class="load-customize hide-if-no-js" ' . $customize_attributes . '>' . __( 'Customize' ) . '</a>';
             if ( ! is_multisite() && current_user_can( 'delete_themes' ) )
-                $actions[] = '<a class="submitdelete deletion" href="' . wp_nonce_url( "themes.php?action=delete&amp;template=$stylesheet", 'delete-theme_' . $stylesheet )
+                $actions['delete'] = '<a class="submitdelete deletion" href="' . wp_nonce_url( "themes.php?action=delete&amp;template=$stylesheet", 'delete-theme_' . $stylesheet )
                     . '" onclick="' . "return confirm( '" . esc_js( sprintf( __( "You are about to delete this theme '%s'\n  'Cancel' to stop, 'OK' to delete." ), $title ) )
                     . "' );" . '">' . __( 'Delete' ) . '</a>';
 
-            $actions = apply_filters( 'theme_action_links', $actions, $theme );
-
-            $actions = implode ( ' | ', $actions );
+            $actions       = apply_filters( 'theme_action_links', $actions, $theme );
+            $delete_action = isset( $actions['delete'] ) ? '<div class="delete-theme">' . $actions['delete'] . '</div>' : '';
+            unset( $actions['delete'] );
+
             ?>
             <a href="<?php echo $preview_link; ?>" class="load-customize screenshot" <?php echo $customize_attributes; ?>>
@@ -158 +159 @@
             <div class="theme-author"><?php printf( __( 'By %s' ), $author ); ?></div>
             <div class="action-links">
-                <?php echo $actions; ?>
-                <span class="separator hide-if-no-js">| </span><a href="#" class="theme-detail hide-if-no-js" tabindex='4'><?php _e('Details') ?></a>
+                <ul>
+                    <?php foreach ( $actions as $action ): ?>
+                        <li><?php echo $action; ?></li>
+                    <?php endforeach; ?>
+                    <li class="hide-if-no-js"><a href="#" class="theme-detail" tabindex='4'><?php _e('Details') ?></a></li>
+                </ul>
+                <?php echo $delete_action; ?>
+
+                <?php theme_update_available( $theme ); ?>
             </div>
 
@@ -176 +184 @@
             </div>
 
-            <?php theme_update_available( $theme ); ?>
-
             </div>
         <?php