WordPress.org

Make WordPress Core

Ticket #21562: 21562.8.diff

File 21562.8.diff, 4.9 KB (added by lancewillett, 8 years ago)

Simpler regex check for class value, add back in the empty menu check, can remove once core fix is in

  • wp-content/themes/twentytwelve/style.css

     
    513513        margin-top: 1.714285714rem;
    514514        text-align: center;
    515515}
    516 .main-navigation li ul ul {
     516.main-navigation li ul {
    517517        margin-left: 40px;
    518518        margin-left: 2.857142857rem;
    519519}
     
    531531.main-navigation a:hover {
    532532        color: #21759b;
    533533}
    534 .main-navigation .menu {
     534.main-navigation ul.nav-menu,
     535.main-navigation div.nav-menu > ul {
    535536        display: none;
    536537}
     538.main-navigation ul.nav-menu.toggled-on,
    537539.menu-toggle {
    538540        display: inline-block;
    539541}
     
    13401342                font-size: 1.857142857rem;
    13411343                line-height: 1.846153846;
    13421344        }
    1343         .main-navigation .menu {
     1345        .main-navigation ul.nav-menu,
     1346        .main-navigation div.nav-menu > ul {
    13441347                border-bottom: 1px solid #ededed;
    13451348                border-top: 1px solid #ededed;
    13461349                display: inline-block !important;
     
    13531356                margin: 0;
    13541357                text-indent: 0;
    13551358        }
    1356         .main-navigation li ul li {
    1357                 padding-left: 0;
    1358         }
    13591359        .main-navigation li a,
    13601360        .main-navigation li {
    13611361                display: inline-block;
    13621362                text-decoration: none;
    13631363        }
    13641364        .main-navigation li a {
     1365                border-bottom: 0;
    13651366                color: #6a6a6a;
     1367                line-height: 3.692307692;
    13661368                text-transform: uppercase;
    13671369        }
    13681370        .main-navigation li a:hover {
     
    13731375                margin: 0 2.857142857rem 0 0;
    13741376                position: relative;
    13751377        }
    1376         .main-navigation li a {
    1377                 border-bottom: 0;
    1378                 line-height: 3.692307692;
    1379         }
    13801378        .main-navigation li ul {
    13811379                display: none;
    13821380                margin: 0;
     
    13861384                z-index: 1;
    13871385        }
    13881386        .main-navigation li ul ul {
    1389                 margin-left: 0;
    13901387                top: 0;
    13911388                left: 100%;
    13921389        }
     
    15481545        .site-content nav,
    15491546        .widget-area,
    15501547        img.header-image,
    1551         nav.site-navigation {
     1548        .main-navigation {
    15521549                display: none;
    15531550        }
    15541551        .wrapper {
  • wp-content/themes/twentytwelve/js/navigation.js

     
    33 *
    44 * Handles toggling the navigation menu for small screens.
    55 */
     6( function() {
     7    var button = document.getElementById( 'site-navigation' ).getElementsByTagName( 'h3' )[0],
     8        menu   = document.getElementById( 'site-navigation' ).getElementsByTagName( 'ul' )[0];
    69
    7 jQuery( document ).ready( function( $ ) {
    8         if ( ! $( '#masthead .menu' ).children().length ) {
    9                 $( '#masthead .menu-toggle' ).hide();
    10         }
     10    button.onclick = function() {
     11        if ( -1 == menu.className.indexOf( 'nav-menu' ) )
     12            menu.className = 'nav-menu';
    1113
    12         $( '.menu-toggle' ).off( 'click' ).click( function() {
    13                 $( '#masthead .menu' ).stop().slideToggle();
    14                 $( this ).toggleClass( 'toggled-on' );
    15         } );
    16 } );
    17  No newline at end of file
     14        if ( -1 != button.className.indexOf( 'toggled-on' ) ) {
     15            button.className = button.className.replace( ' toggled-on', '' );
     16            menu.className = menu.className.replace( ' toggled-on', '' );
     17        } else {
     18            button.className += ' toggled-on';
     19            menu.className += ' toggled-on';
     20        }
     21    };
     22
     23    // Hide menu toggle button if menu is empty.
     24    if ( ! menu.childNodes.length )
     25        button.style.display = 'none';
     26} )();
     27 No newline at end of file
  • wp-content/themes/twentytwelve/header.php

     
    3535                        <h2 class="site-description"><?php bloginfo( 'description' ); ?></h2>
    3636                </hgroup>
    3737
    38                 <nav class="site-navigation main-navigation" role="navigation">
     38                <nav id="site-navigation" class="main-navigation" role="navigation">
    3939                        <h3 class="menu-toggle"><?php _e( 'Show menu', 'twentytwelve' ); ?></h3>
    4040                        <div class="skip-link assistive-text"><a href="#content" title="<?php esc_attr_e( 'Skip to content', 'twentytwelve' ); ?>"><?php _e( 'Skip to content', 'twentytwelve' ); ?></a></div>
    41                         <?php wp_nav_menu( array( 'theme_location' => 'primary' ) ); ?>
     41                        <?php wp_nav_menu( array( 'theme_location' => 'primary', 'menu_class' => 'nav-menu' ) ); ?>
    4242                </nav>
    4343
    4444                <?php $header_image = get_header_image();
  • wp-content/themes/twentytwelve/functions.php

     
    104104         * JavaScript for handling navigation menus and the resized
    105105         * styles for small screen sizes.
    106106         */
    107         wp_enqueue_script( 'twentytwelve-navigation', get_template_directory_uri() . '/js/navigation.js', array( 'jquery' ), '20120820', true );
     107        wp_enqueue_script( 'twentytwelve-navigation', get_template_directory_uri() . '/js/navigation.js', array(), '20120820', true );
    108108
    109109        /**
    110110         * Load special font CSS file.