WordPress.org

Make WordPress Core

Ticket #12542: theme_h1_and_title_fixes.diff

File theme_h1_and_title_fixes.diff, 3.0 KB (added by mikeschinkel, 5 years ago)

Patch to resolve <title> and <h1> issues with Twenty Ten theme.

  • functions.php

     
    257257} 
    258258endif; 
    259259add_action( 'init', 'twentyten_widgets_init' ); 
     260 
     261function is_root_page() { 
     262        return (is_front_page() || (is_front_page() && is_home())); 
     263} 
     264function the_header_title_element() { 
     265        echo (is_root_page() ? 'h1' : 'div'); 
     266} 
     267 No newline at end of file 
  • onecolumn-page.php

     
    1212<?php the_post(); ?> 
    1313 
    1414                                <div id="post-<?php the_ID(); ?>" <?php post_class(); ?>> 
     15                                        <?php if (!is_root_page()): ?> 
    1516                                        <h1 class="entry-title"><?php the_title(); ?></h1> 
     17                                        <?php endif; ?> 
    1618                                        <div class="entry-content"> 
    1719                                                <?php the_content(); ?> 
    1820                                                <?php wp_link_pages( 'before=<div class="page-link">' . __( 'Pages:', 'twentyten' ) . '&after=</div>'); ?> 
  • page.php

     
    66<?php the_post(); ?> 
    77 
    88                                <div id="post-<?php the_ID(); ?>" <?php post_class(); ?>> 
     9                                        <?php if (!is_root_page()): ?> 
    910                                        <h1 class="entry-title"><?php the_title(); ?></h1> 
     11                                        <?php endif; ?> 
    1012                                        <div class="entry-content"> 
    1113                                                <?php the_content(); ?> 
    1214                                                <?php wp_link_pages( 'before=<div class="page-link">' . __( 'Pages:', 'twentyten' ) . '&after=</div>'); ?> 
  • header.php

     
    33<head> 
    44        <meta charset="<?php bloginfo('charset'); ?>" /> 
    55    <title><?php 
    6         if ( is_single() ) { 
     6    if ( is_single() ) { 
    77                        single_post_title(); echo ' | '; bloginfo('name'); 
    8                 } elseif ( is_home() || is_front_page() ) { 
    9                         bloginfo('name'); echo ' | '; bloginfo('description'); twentyten_the_page_number(); 
     8    } elseif ( is_root_page() ) { 
     9            bloginfo('name'); echo ' | '; bloginfo('description'); twentyten_the_page_number(); 
    1010                } elseif ( is_page() ) { 
    1111                        single_post_title(''); echo ' | '; bloginfo('name'); 
    1212                } elseif ( is_search() ) { 
     
    2525        <?php wp_head(); ?> 
    2626</head> 
    2727 
     28 
    2829<body <?php body_class(); ?>> 
    2930<div id="wrapper" class="hfeed"> 
    3031        <div id="header"> 
    3132                <div id="masthead"> 
    3233                        <div id="branding"> 
    33                                 <div id="site-title"><span><a href="<?php echo home_url('/'); ?>" title="<?php echo esc_attr( get_bloginfo( 'name', 'display' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a></span></div> 
     34                                <<?php the_header_title_element(); ?> id="site-title"><span><a href="<?php echo home_url('/'); ?>" title="<?php echo esc_attr( get_bloginfo( 'name', 'display' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a></span></<?php the_header_title_element(); ?>> 
    3435                                <div id="site-description"><?php bloginfo( 'description' ); ?></div> 
    3536 
    3637                                <?php 
     
    4950        </div><!-- #header --> 
    5051 
    5152        <div id="main"> 
     53 
     54<?php 
     55