WordPress.org
Get WordPress

Make WordPress Core

Ticket #21413: 21413.2.diff

File 21413.2.diff, 7.3 KB (added by DrewAPicture, 12 years ago)

Refreshed and cleanup on themes.php

  • wp-admin/admin-header.php

     
    9898</script>
    9999
    100100<?php
    101 // If the customize-loader script is enqueued, make sure the customize
    102 // body classes are correct as early as possible.
    103 if ( wp_script_is( 'customize-loader', 'queue' ) && current_user_can( 'edit_theme_options' ) )
     101// Make sure the customize body classes are correct as early as possible.
     102if (  current_user_can( 'edit_theme_options' ) )
    104103        wp_customize_support_script();
    105104?>
    106105

  • wp-admin/css/wp-admin-rtl.css

     
    13051305        margin-right: -330px;
    13061306}
    13071307
    1308 .theme-options .load-customize {
    1309         margin-right: 0;
    1310         margin-left: 30px;
    1311         float: right;
    1312 }
    1313 
    13141308.theme-options span {
    13151309        float: right;
    13161310        margin-right: 0;
    13171311        margin-left: 10px;
    13181312}
    13191313
    1320 .theme-options ul {
     1314.theme-options ul,
     1315.theme-options .load-customize {
    13211316        float: right;
    13221317}
    13231318

  • wp-admin/css/wp-admin.css

     
    47594759        padding-bottom: 10px;
    47604760}
    47614761
    4762 .theme-options .load-customize {
    4763         margin-right: 30px;
    4764         float: left;
    4765 }
    4766 
    47674762.theme-options span {
    47684763        float: left;
    47694764        margin-right: 10px;
    … …  
    47734768        color: #999;
    47744769}
    47754770
    4776 .theme-options ul {
     4771.theme-options ul,
     4772.theme-options .load-customize {
    47774773        float: left;
    47784774        margin: 0;
    47794775}

  • wp-admin/includes/menu.php

     
    7373        $new_parent = $first_sub[2];
    7474        // If the first submenu is not the same as the assigned parent,
    7575        // make the first submenu the new parent.
    76         if ( $new_parent != $old_parent ) {
     76        if ( $new_parent != $old_parent && 'customize.php' != $new_parent ) {
    7777                $_wp_real_parent_file[$old_parent] = $new_parent;
    7878                $menu[$id][2] = $new_parent;
    7979

  • wp-admin/menu-header.php

     
    5353                        $class[] = 'wp-has-submenu';
    5454                        $submenu_items = $submenu[$item[2]];
    5555                }
    56 
     56               
    5757                if ( ( $parent_file && $item[2] == $parent_file ) || ( empty($typenow) && $self == $item[2] ) ) {
    5858                        $class[] = ! empty( $submenu_items ) ? 'wp-has-current-submenu wp-menu-open' : 'current';
    5959                } else {
    … …  
    8282                if ( false !== strpos( $class, 'wp-menu-separator' ) ) {
    8383                        echo '<div class="separator"></div>';
    8484                } elseif ( $submenu_as_parent && ! empty( $submenu_items ) ) {
    85                         $submenu_items = array_values( $submenu_items );  // Re-index.
     85                        $temp_submenu_items = $submenu_items = array_values( $submenu_items );  // Re-index.
     86                        if ( 'customize.php' == $submenu_items[0][2] )
     87                                $submenu_items[0][2] = 'themes.php';
    8688                        $menu_hook = get_plugin_page_hook( $submenu_items[0][2], $item[2] );
    8789                        $menu_file = $submenu_items[0][2];
    8890                        if ( false !== ( $pos = strpos( $menu_file, '?' ) ) )
    … …  
    9395                        } else {
    9496                                echo "\n\t<a href='{$submenu_items[0][2]}'$class $aria_attributes>$arrow<div class='wp-menu-image'>$img</div><div class='wp-menu-name'>$title</div></a>";
    9597                        }
     98                        $submenu_items = $temp_submenu_items;
    9699                } elseif ( ! empty( $item[2] ) && current_user_can( $item[1] ) ) {
    97100                        $menu_hook = get_plugin_page_hook( $item[2], 'admin.php' );
    98101                        $menu_file = $item[2];
    … …  
    105108                                echo "\n\t<a href='{$item[2]}'$class $aria_attributes>$arrow<div class='wp-menu-image'>$img</div><div class='wp-menu-name'>{$item[0]}</div></a>";
    106109                        }
    107110                }
    108 
     111               
    109112                if ( ! empty( $submenu_items ) ) {
    110113                        echo "\n\t<ul class='wp-submenu wp-submenu-wrap'>";
    111114                        echo "<li class='wp-submenu-head'>{$item[0]}</li>";
    … …  
    140143                                ) {
    141144                                        $class[] = 'current';
    142145                                }
     146                               
     147                                if ( ! empty( $sub_item[3] ) )
     148                                        $class[] = $sub_item[3];
    143149
    144150                                $class = $class ? ' class="' . join( ' ', $class ) . '"' : '';
    145151

  • wp-admin/menu.php

     
    136136
    137137$menu[59] = array( '', 'read', 'separator2', '', 'wp-menu-separator' );
    138138
    139 if ( current_user_can( 'switch_themes') ) {
    140         $menu[60] = array( __('Appearance'), 'switch_themes', 'themes.php', '', 'menu-top menu-icon-appearance', 'menu-appearance', 'none' );
    141                 $submenu['themes.php'][5]  = array(__('Themes'), 'switch_themes', 'themes.php');
    142                 if ( current_theme_supports( 'menus' ) || current_theme_supports( 'widgets' ) )
    143                         $submenu['themes.php'][10] = array(__('Menus'), 'edit_theme_options', 'nav-menus.php');
    144 } else {
    145         $menu[60] = array( __('Appearance'), 'edit_theme_options', 'themes.php', '', 'menu-top menu-icon-appearance', 'menu-appearance', 'none' );
    146                 $submenu['themes.php'][5]  = array(__('Themes'), 'edit_theme_options', 'themes.php');
    147                 if ( current_theme_supports( 'menus' ) || current_theme_supports( 'widgets' ) )
    148                         $submenu['themes.php'][10] = array(__('Menus'), 'edit_theme_options', 'nav-menus.php' );
    149 }
     139$appearance_cap = current_user_can( 'switch_themes') ? 'switch_themes' : 'edit_theme_options';
     140 
     141$menu[60] = array( __('Appearance'), $appearance_cap, 'themes.php', '', 'menu-top menu-icon-appearance', 'menu-appearance', 'div' );
     142        $submenu['themes.php'][4] = array( __( 'Customize' ), $appearance_cap, 'customize.php', 'hide-if-no-customize' );
     143        $submenu['themes.php'][5] = array(__( 'Themes' ), $appearance_cap, 'themes.php');
     144        if ( current_theme_supports( 'menus' ) || current_theme_supports( 'widgets' ) )
     145                $submenu['themes.php'][10] = array(__( 'Menus' ), 'edit_theme_options', 'nav-menus.php');
    150146
     147unset( $appearance_cap );
     148
    151149// Add 'Editor' to the bottom of the Appearance menu.
    152150if ( ! is_multisite() )
    153151        add_action('admin_menu', '_add_themes_utility_last', 101);

  • wp-admin/themes.php

     
    161161                        $class = '';
    162162                        if ( 'themes.php' == $item[2] || 'theme-editor.php' == $item[2] )
    163163                                continue;
     164                        if ( 'customize.php' == $item[2] ) {
     165                                if ( current_user_can( 'edit_theme_options' ) )
     166                                        $class = ' class="load-customize hide-if-no-customize"';
     167                                else
     168                                        continue;
     169                        }
    164170                        // 0 = name, 1 = capability, 2 = file
    165171                        if ( ( strcmp($self, $item[2]) == 0 && empty($parent_file)) || ($parent_file && ($item[2] == $parent_file)) )
    166172                                $class = ' class="current"';
    … …  
    187193        if ( $options || current_user_can( 'edit_theme_options' ) ) :
    188194        ?>
    189195        <div class="theme-options">
    190                 <?php if ( current_user_can( 'edit_theme_options' ) ) : ?>
    191                 <a id="customize-current-theme-link" href="<?php echo wp_customize_url(); ?>" class="load-customize hide-if-no-customize" title="<?php echo esc_attr( $customize_title ); ?>"><?php _e( 'Customize' ); ?></a>
    192                 <?php
    193                 endif; // edit_theme_options
    194                 if ( $options ) :
    195                 ?>
     196                <?php if ( $options ) : ?>
    196197                <span><?php _e( 'Options:' )?></span>
    197198                <ul>
    198199                        <?php foreach ( $options as $option ) : ?>