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, 8 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