WordPress.org

Make WordPress Core

Ticket #13751: 13751.2.diff

File 13751.2.diff, 9.3 KB (added by nacin, 3 years ago)

  • wp-includes/theme.php

     
    15351535function _custom_background_cb() { 
    15361536        $background = get_background_image(); 
    15371537        $color = get_background_color(); 
    1538         if ( !$background && !$color ) 
     1538        if ( ! $background && ! $color ) 
    15391539                return; 
    15401540 
    1541         switch ( get_theme_mod('background_repeat', 'repeat') ) { 
    1542                 case 'no-repeat': 
    1543                         $repeat = 'background-repeat: no-repeat;'; 
    1544                         break; 
    1545                 case 'repeat-x': 
    1546                         $repeat = 'background-repeat: repeat-x;'; 
    1547                         break; 
    1548                 case 'repeat-y': 
    1549                         $repeat = 'background-repeat: repeat-y;'; 
    1550                         break; 
    1551                 default: 
    1552                         $repeat = 'background-repeat: repeat;'; 
    1553         } 
     1541        $style = $color ? "background-color: #$color;" : ''; 
    15541542 
    1555         switch ( get_theme_mod('background_position_x', 'left') ) { 
    1556                 case 'center': 
    1557                         $position = 'background-position: top center;'; 
    1558                         break; 
    1559                 case 'right': 
    1560                         $position = 'background-position: top right;'; 
    1561                         break; 
    1562                 default: 
    1563                         $position = 'background-position: top left;'; 
    1564         } 
     1543        if ( $background ) { 
     1544                $image = " background-image: url('$background');"; 
    15651545 
    1566         if ( 'scroll' == get_theme_mod('background_attachment', 'fixed') ) 
    1567                 $attachment = 'background-attachment: scroll;'; 
    1568         else 
    1569                 $attachment = 'background-attachment: fixed;'; 
     1546                $repeat = get_theme_mod( 'background_repeat', 'repeat' ); 
     1547                if ( ! in_array( $repeat, array( 'no-repeat', 'repeat-x', 'repeat-y', 'repeat' ) ) ) 
     1548                        $repeat = 'repeat'; 
     1549                $repeat = " background-repeat: $repeat;"; 
     1550         
     1551                $position = get_theme_mod( 'background_position_x', 'left' ); 
     1552                if ( ! in_array( $position, array( 'center', 'right', 'left' ) ) ) 
     1553                        $position = 'left'; 
     1554                $position = " background-position: top $position;"; 
     1555         
     1556                $attachment = get_theme_mod( 'background_attachment', 'scroll' ); 
     1557                if ( ! in_array( $attachment, array( 'fixed', 'scroll' ) ) ) 
     1558                        $attachment = 'scroll'; 
     1559                $attachment = " background-attachment: $attachment;"; 
    15701560 
    1571         if ( !empty($background ) ) 
    1572                 $image = "background-image: url('$background');"; 
    1573         else 
    1574                 $image = ''; 
    1575  
    1576         if ( !empty($color) ) 
    1577                 $color = "background-color: #$color;"; 
    1578         else 
    1579                 $color = ''; 
     1561                $style .= $image . $repeat . $position . $attachment; 
     1562        } 
    15801563?> 
    15811564<style type="text/css"> 
    1582 body { 
    1583         <?php echo $image; ?> 
    1584         <?php echo $color; ?> 
    1585         <?php echo $repeat; ?> 
    1586         <?php echo $position; ?> 
    1587         <?php echo $attachment; ?> 
    1588 } 
     1565body { <?php echo trim( $style ); ?> } 
    15891566</style> 
    15901567<?php 
    15911568} 

  • wp-content/themes/twentyten/functions.php

     
    190190<style type="text/css"> 
    191191/* Shows the same border as on front end */ 
    192192#headimg { 
    193         border-bottom: 1px solid #000000; 
    194         border-top: 4px solid #000000; 
     193        border-bottom: 1px solid #000; 
     194        border-top: 4px solid #000; 
    195195} 
    196  
    197 /* If NO_HEADER_TEXT is false, you can style here the header text preview */ 
    198 #headimg #name { 
    199 } 
    200  
    201 #headimg #desc { 
    202 } 
     196/* If NO_HEADER_TEXT is false, you would style the text with these selectors: 
     197        #headimg #name { } 
     198        #headimg #desc { } 
     199*/ 
    203200</style> 
    204201<?php 
    205202} 
    206203endif; 
    207204 
    208 if ( ! function_exists( 'twentyten_the_page_number' ) ) : 
    209205/** 
    210  * Prints the page number currently being browsed, with a vertical bar before it. 
     206 * Makes some changes to the <title> tag, by filtering the output of wp_title(). 
    211207 * 
    212  * Used in Twenty Ten's header.php to add the page number to the <title> HTML tag. 
     208 * If we have a site description and we're viewing the home page or a blog posts 
     209 * page (when using a static front page), then we will add the site description. 
    213210 * 
     211 * If we're viewing a search result, then we're going to recreate the title entirely. 
     212 * We're going to add page numbers to all titles as well, to the middle of a search 
     213 * result title and the end of all other titles. 
     214 * 
     215 * The site title also gets added to all titles. 
     216 * 
    214217 * @since Twenty Ten 1.0 
     218 * 
     219 * @param string $title Title generated by wp_title() 
     220 * @param string $separator The separator passed to wp_title(). Twenty Ten uses a 
     221 *      vertical bar, "|", as a separator in header.php. 
     222 * @return string The new title, ready for the <title> tag. 
    215223 */ 
    216 function twentyten_the_page_number() { 
    217         global $paged; // Contains page number. 
    218         if ( $paged >= 2 ) 
    219                 echo ' | ' . sprintf( __( 'Page %s', 'twentyten' ), $paged ); 
     224function twentyten_filter_wp_title( $title, $separator ) { 
     225        // The $paged global variable contains the page number of a listing of posts. 
     226        // The $page global variable contains the page number of a single post that is paged. 
     227        // We'll display whichever one applies, if we're not looking at the first page. 
     228        global $paged, $page; 
     229 
     230        if ( is_search() ) { 
     231                // If we're a search, let's start over: 
     232                $title = sprintf( __( 'Search results for %s', 'twentyten' ), '"' . get_search_query() . '"' ); 
     233                // Add a page number if we're on page 2 or more: 
     234                if ( $paged >= 2 ) 
     235                        $title .= " $separator " . sprintf( __( 'Page %s', 'twentyten' ), $paged ); 
     236                // Add the site name to the end: 
     237                $title .= " $separator " . get_bloginfo( 'name', 'display' ); 
     238                // We're done. Let's send the new title back to wp_title(): 
     239                return $title; 
     240        } 
     241 
     242        // Otherwise, let's start by adding the site name to the end: 
     243        $title .= get_bloginfo( 'name', 'display' ); 
     244 
     245        // If we have a site description and we're on the home/front page, add the description: 
     246        $site_description = get_bloginfo( 'description', 'display' ); 
     247        if ( $site_description && ( is_home() || is_front_page() ) ) 
     248                $title .= " $separator " . $site_description; 
     249 
     250        // Add a page number if necessary: 
     251        if ( $paged >= 2 || $page >= 2 ) 
     252                $title .= " $separator " . sprintf( __( 'Page %s', 'twentyten' ), max( $paged, $page ) ); 
     253 
     254        // Return the new title to wp_title(): 
     255        return $title; 
    220256} 
    221 endif; 
     257add_filter( 'wp_title', 'twentyten_filter_wp_title', 10, 2 ); 
    222258 
    223259/** 
    224260 * Get our wp_nav_menu() fallback, wp_page_menu(), to show a home link. 

  • wp-content/themes/twentyten/header.php

     
    1111?><!DOCTYPE html> 
    1212<html <?php language_attributes(); ?>> 
    1313<head> 
    14         <meta charset="<?php bloginfo( 'charset' ); ?>" /> 
    15         <title> 
    16         <?php // Returns the title based on what is being viewed 
    17                 if ( is_single() ) { // single posts 
    18                         single_post_title(); echo ' | '; bloginfo( 'name' ); 
    19                 // The home page or, if using a static front page, the blog posts page. 
    20                 } elseif ( is_home() || is_front_page() ) { 
    21                         bloginfo( 'name' ); 
    22                         if( get_bloginfo( 'description' ) ) 
    23                                 echo ' | ' ; bloginfo( 'description' ); 
    24                         twentyten_the_page_number(); 
    25                 } elseif ( is_page() ) { // WordPress Pages 
    26                         single_post_title( '' ); echo ' | '; bloginfo( 'name' ); 
    27                 } elseif ( is_search() ) { // Search results 
    28                         printf( __( 'Search results for %s', 'twentyten' ), '"'.get_search_query().'"' ); twentyten_the_page_number(); echo ' | '; bloginfo( 'name' ); 
    29                 } elseif ( is_404() ) {  // 404 (Not Found) 
    30                         _e( 'Not Found', 'twentyten' ); echo ' | '; bloginfo( 'name' ); 
    31                 } else { // Otherwise: 
    32                         wp_title( '' ); echo ' | '; bloginfo( 'name' ); twentyten_the_page_number(); 
    33                 } 
    34         ?> 
    35         </title> 
    36         <link rel="profile" href="http://gmpg.org/xfn/11" /> 
    37         <link rel="stylesheet" type="text/css" media="all" href="<?php bloginfo( 'stylesheet_url' ); ?>" /> 
    38         <link rel="pingback" href="<?php bloginfo( 'pingback_url' ); ?>" /> 
     14<meta charset="<?php bloginfo( 'charset' ); ?>" /> 
     15<title><?php 
     16        /* 
     17         * Print the <title> tag based on what is being viewed. 
     18         * We filter the output of wp_title() a bit -- see 
     19         * twentyten_filter_wp_title() in functions.php. 
     20         */ 
     21        wp_title( '|', true, 'right' ); 
     22 
     23        ?></title> 
     24<link rel="profile" href="http://gmpg.org/xfn/11" /> 
     25<link rel="stylesheet" type="text/css" media="all" href="<?php bloginfo( 'stylesheet_url' ); ?>" /> 
     26<link rel="pingback" href="<?php bloginfo( 'pingback_url' ); ?>" /> 
    3927<?php 
    4028        /* We add some JavaScript to pages with the comment form 
    4129         * to support sites with threaded comments (when in use). 
    … …  
    4836         * generally use this hook to add elements to <head> such 
    4937         * as styles, scripts, and meta tags. 
    5038         */ 
    51  
    5239        wp_head(); 
    5340?> 
    5441</head> 

  • wp-admin/custom-background.php

     
    281281<th scope="row"><?php _e( 'Attachment' ); ?></th> 
    282282<td><fieldset><legend class="screen-reader-text"><span><?php _e( 'Background Attachment' ); ?></span></legend> 
    283283<label> 
    284 <input name="background-attachment" type="radio" value="scroll" <?php checked('scroll', get_theme_mod('background_attachment', 'fixed')); ?> /> 
     284<input name="background-attachment" type="radio" value="scroll" <?php checked('scroll', get_theme_mod('background_attachment', 'scroll')); ?> /> 
    285285<?php _e('Scroll') ?> 
    286286</label> 
    287287<label> 
    288 <input name="background-attachment" type="radio" value="fixed" <?php checked('fixed', get_theme_mod('background_attachment', 'fixed')); ?> /> 
     288<input name="background-attachment" type="radio" value="fixed" <?php checked('fixed', get_theme_mod('background_attachment', 'scroll')); ?> /> 
    289289<?php _e('Fixed') ?> 
    290290</label> 
    291291</fieldset></td>