Changeset 30567

Timestamp:

11/25/2014 03:11:11 PM (10 years ago)

Author:

iandstewart

Message:

Twenty Fifteen: remove unneeded custom header admin callbacks.

Props philiparthurmoore, fixes #30432.

Location:

trunk/src/wp-content/themes/twentyfifteen

Files:

• css/admin-custom-header.css (modified) (1 diff)
• inc/custom-header.php (modified) (5 diffs)

trunk/src/wp-content/themes/twentyfifteen/css/admin-custom-header.css

@@ -1 @@
-/*
-Theme Name: Twenty Fifteen
-Description: Used to style the header image displayed on the Appearance > Header screen.
-*/
-
-.appearance_page_custom-header #headimg {
-    background: #fff no-repeat 50% 50%;
-    background-size: cover;
-    border: none;
-    box-sizing: border-box;
-    padding: 7.6923%;
-}
-
-#headimg h1,
-#desc {
-    color: #333;
-    font-family: "Noto Sans", sans-serif;
-}
-
-#headimg h1 {
-    font-size: 22px;
-    line-height: 1.0909;
-    margin: 0;
-}
-
-#headimg h1 a {
-    text-decoration: none;
-}
-
-#desc {
-    font-size: 12px;
-    line-height: 1.5;
-    margin-top: 0.5em;
-    opacity: 0.7;
-}
-
-/* Hide the header text color option */
-.appearance_page_custom-header tr.displaying-header-text,
-.appearance_page_custom-header tr.displaying-header-text td,
-.appearance_page_custom-header tr.displaying-header-text th {
-    display: none;
-}
-
-@media screen and (min-width: 772px) {
-    #headimg h1 {
-        font-size: 29px;
-        line-height: 1.2069;
-        margin: 0;
-    }
-
-    #desc {
-        font-size: 14px;
-    }
-}
-
-@media screen and (min-width: 783px) {
-    #headimg h1 {
-        font-size: 22px;
-        line-height: 1.0909;
-    }
-
-    #desc {
-        font-size: 12px;
-        line-height: 1.5;
-        margin-top: 0.5em;
-    }
-}
-
-@media screen and (min-width: 1172px) {
-    #headimg h1 {
-        font-size: 29px;
-        line-height: 1.2069;
-    }
-
-    #desc {
-        font-size: 14px;
-    }
-}
-
-@media screen and (min-width: 1313px) {
-    #headimg h1 {
-        font-size: 32px;
-        line-height: 1.25;
-    }
-
-    #desc {
-        font-size: 16px;
-    }
-}
-
-@media screen and (min-width: 1383px) {
-    .appearance_page_custom-header #headimg {
-        background-position: 100% 50%;
-        height: 1300px;
-        max-width: 954px;
-        padding: 83px 83px 0 0;
-        width: 42%;
-    }
-
-    #headimg h1 {
-        float: right;
-        font-size: 27px;
-        line-height: 1.1852;
-        width: 248px;
-    }
-
-    #desc {
-        clear: right;
-        float: right;
-        width: 248px;
-    }
-}

trunk/src/wp-content/themes/twentyfifteen/inc/custom-header.php

@@ -12 +12 @@
  *
  * @uses twentyfifteen_header_style()
- * @uses twentyfifteen_admin_header_image()
  */
 function twentyfifteen_custom_header_setup() {
@@ -31 +30 @@
      *     @type string $wp-head-callback       Callback function used to styles the header image and text
      *                                          displayed on the blog.
-     *     @type string $admin-preview-callback Callback function used to create the custom header markup in
-     *                                          the Appearance > Header screen.
      * }
      */
@@ -40 +37 @@
         'height'                 => 1300,
         'wp-head-callback'       => 'twentyfifteen_header_style',
-        'admin-preview-callback' => 'twentyfifteen_admin_header_image',
     ) ) );
 }
@@ -173 +169 @@
 }
 endif; // twentyfifteen_header_style
-
-if ( ! function_exists( 'twentyfifteen_admin_header_image' ) ) :
-/**
- * Custom header image markup displayed on the Appearance > Header admin panel.
- *
- * @since Twenty Fifteen 1.0
- * @see twentyfifteen_custom_header_setup().
- */
-function twentyfifteen_admin_header_image() {
-    $style                   = sprintf( ' style="color: #%s;"', esc_attr( get_header_textcolor() ) );
-    $color_scheme            = twentyfifteen_get_color_scheme();
-    $header_background_color = get_theme_mod( 'header_background_color', $color_scheme[1] );
-?>
-    <div id="headimg" style="background-image: url(<?php header_image(); ?>); background-color: <?php echo esc_attr( $header_background_color ); ?>;">
-        <h1 class="displaying-header-text"><a id="name"<?php echo $style; ?> onclick="return false;" href="<?php echo esc_url( home_url( '/' ) ); ?>" tabindex="-1"><?php bloginfo( 'name' ); ?></a></h1>
-        <div id="desc" class="displaying-header-text"<?php echo $style; ?>><?php bloginfo( 'description' ); ?></div>
-    </div>
-<?php
-}
-endif; // twentyfifteen_admin_header_image
 
 /**
@@ -375 +351 @@
 }
 add_action( 'wp_enqueue_scripts', 'twentyfifteen_sidebar_text_color_css', 11 );
-
-/**
- * Enqueue styles to admin screen for custom header display.
- *
- * @since Twenty Fifteen 1.0
- */
-function twentyfifteen_admin_fonts() {
-    wp_enqueue_style( 'twentyfifteen-fonts', twentyfifteen_fonts_url(), array(), null );
-    wp_enqueue_style( 'twentyfifteen-custom-header', get_template_directory_uri() . '/css/admin-custom-header.css', array(), '20141010' );
-}
-add_action( 'admin_print_scripts-appearance_page_custom-header', 'twentyfifteen_admin_fonts' );