WordPress.org

Make WordPress Core

Ticket #22307: 22307.2.diff

File 22307.2.diff, 1.2 KB (added by nacin, 18 months ago)

Perhaps a nicer way to do this.

  • 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]; 
     6(function() { 
     7        var nav = document.getElementById( 'site-navigation' ), button, menu; 
     8        if ( ! nav ) 
     9                return; 
     10        button = nav.getElementsByTagName( 'h3' )[0]; 
     11        menu   = nav.getElementsByTagName( 'ul' )[0]; 
     12        if ( ! button ) 
     13                return; 
    914 
    10         if ( undefined === button ) 
    11                 return false; 
    12  
    1315        // Hide button if menu is missing or empty. 
    14         if ( undefined === menu || ! menu.childNodes.length ) { 
     16        if ( ! menu || ! menu.childNodes.length ) { 
    1517                button.style.display = 'none'; 
    16                 return false; 
     18                return; 
    1719        } 
    1820 
    1921        button.onclick = function() { 
     
    2830                        menu.className += ' toggled-on'; 
    2931                } 
    3032        }; 
    31 } )(); 
    32  No newline at end of file 
     33})(); 
     34 No newline at end of file