Ticket #13751: 13751.diff

File 13751.diff, 6.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} 

  • 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' ); 
     14<meta charset="<?php bloginfo( 'charset' ); ?>" /> 
     15<title><?php 
     16                // Returns the title based on what is being viewed. 
    1917                // The home page or, if using a static front page, the blog posts page. 
    20                 } elseif ( is_home() || is_front_page() ) { 
     18                if ( is_home() || is_front_page() ) { 
    2119                        bloginfo( 'name' ); 
    2220                        if( get_bloginfo( 'description' ) ) 
    2321                                echo ' | ' ; bloginfo( 'description' ); 
    2422                        twentyten_the_page_number(); 
    25                 } elseif ( is_page() ) { // WordPress Pages 
    26                         single_post_title( '' ); echo ' | '; bloginfo( 'name' ); 
    2723                } elseif ( is_search() ) { // Search results 
    2824                        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' ); 
    3125                } else { // Otherwise: 
    32                         wp_title( '' ); echo ' | '; bloginfo( 'name' ); twentyten_the_page_number(); 
     26                        wp_title( '|', true, 'right' ); bloginfo( 'name' ); twentyten_the_page_number(); 
    3327                } 
    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' ); ?>" /> 
     28        ?></title> 
     29<link rel="profile" href="http://gmpg.org/xfn/11" /> 
     30<link rel="stylesheet" type="text/css" media="all" href="<?php bloginfo( 'stylesheet_url' ); ?>" /> 
     31<link rel="pingback" href="<?php bloginfo( 'pingback_url' ); ?>" /> 
    3932<?php 
    4033        /* We add some JavaScript to pages with the comment form 
    4134         * to support sites with threaded comments (when in use). 

  • 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>