sergiotarxz
/
Wordpress
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Twenty Thirteen: remove `hgroup` from the theme markup structure since it's been dropped from HTML5 spec. Props obenland for the starter patch. Fixes #24114. 

git-svn-id: https://develop.svn.wordpress.org/trunk@24072 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Lance Willett 2013-04-23 17:55:55 +00:00
parent b051e780e3
commit 736f2bb495
5 changed files with 14 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
wp-content/themes/twentythirteen/css/ie.css
View File

@ -34,7 +34,7 @@ Styles for older IE versions (previous to IE9).
left: 100%;
}
.site-header hgroup {
.site-header .home-link {
max-width: 1040px;
}

8
wp-content/themes/twentythirteen/header.php
View File

@ -39,11 +39,9 @@ $html_class = ( is_admin_bar_showing() ) ? 'wp-toolbar' : '';
<body <?php body_class(); ?>>
<div id="page" class="hfeed site">
<header id="masthead" class="site-header" role="banner">
<a href="<?php echo esc_url( home_url( '/' ) ); ?>" title="<?php echo esc_attr( get_bloginfo( 'name', 'display' ) ); ?>" rel="home">
<hgroup>
<h1 class="site-title"><?php bloginfo( 'name' ); ?></h1>
<h2 class="site-description"><?php bloginfo( 'description' ); ?></h2>
</hgroup>
<a class="home-link" href="<?php echo esc_url( home_url( '/' ) ); ?>" title="<?php echo esc_attr( get_bloginfo( 'name', 'display' ) ); ?>" rel="home">
<h1 class="site-title"><?php bloginfo( 'name' ); ?></h1>
<h2 class="site-description"><?php bloginfo( 'description' ); ?></h2>
</a>
<div id="navbar" class="navbar">

6
wp-content/themes/twentythirteen/inc/custom-header.php
View File

@ -103,7 +103,7 @@ function twentythirteen_header_style() {
<?php
if ( empty( $header_image ) ) :
?>
.site-header hgroup {
.site-header .home-link {
min-height: 0;
}
<?php
@ -141,7 +141,7 @@ function twentythirteen_admin_header_style() {
} ?>
padding: 0 20px;
}
#headimg .hgroup {
#headimg .home-link {
-webkit-box-sizing: border-box;
-moz-box-sizing: border-box;
box-sizing: border-box;
@ -195,7 +195,7 @@ function twentythirteen_admin_header_image() {
?>
<div id="headimg" style="background: url(<?php header_image(); ?>) no-repeat scroll top; background-size: 1600px auto;">
<?php $style = ' style="color:#' . get_header_textcolor() . ';"'; ?>
<div class="hgroup">
<div class="home-link">
<h1 class="displaying-header-text"><a id="name"<?php echo $style; ?> onclick="return false;" href="#"><?php bloginfo( 'name' ); ?></a></h1>
<h2 id="desc" class="displaying-header-text"<?php echo $style; ?>><?php bloginfo( 'description' ); ?></h2>
</div>

4
wp-content/themes/twentythirteen/js/theme-customizer.js
View File

@ -22,13 +22,13 @@
value.bind( function( to ) {
if ( 'blank' == to ) {
if ( 'remove-header' == _wpCustomizeSettings.values.header_image )
$( '#masthead hgroup' ).css( 'min-height', '0' );
$( '.home-link' ).css( 'min-height', '0' );
$( '.site-title, .site-description' ).css( {
'clip': 'rect(1px, 1px, 1px, 1px)',
'position': 'absolute'
} );
} else {
$( '#masthead hgroup' ).css( 'min-height', '230px' );
$( '.home-link' ).css( 'min-height', '230px' );
$( '.site-title, .site-description' ).css( {
'clip': 'auto',
'color': to,

13
wp-content/themes/twentythirteen/style.css
View File

@ -70,7 +70,6 @@ figcaption,
figure,
footer,
header,
hgroup,
nav,
section,
summary {
@ -807,16 +806,14 @@ img.wp-smiley,
position: relative;
}
.site-header hgroup {
.site-header .home-link {
color: #141412;
display: block;
margin: 0 auto;
max-width: 1080px;
padding: 0 20px;
min-height: 230px;
width: 100%;
}
.site-header a {
color: #141412;
text-decoration: none;
}
@ -991,7 +988,7 @@ ul.nav-menu li:hover > ul,
z-index: 3;
}
.navbar-fixed #masthead .site-title {
.navbar-fixed .site-title {
color: #141412;
}
@ -3238,7 +3235,7 @@ footer.entry-meta {
background-image: none !important;
}
.site-header hgroup {
.site-header .home-link {
min-height: 0;
max-width: none;
}