WordPress.org

Make WordPress Core

Ticket #21413: 21413.2.diff

File 21413.2.diff, 7.3 KB (added by DrewAPicture, 3 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 ) : ?>