Changeset 20017

Timestamp:

02/28/2012 07:17:04 PM (13 years ago)

Author:

lancewillett

Message:

Twenty Twelve: make proper use of the cached jQuery collection in navigation.js, props mattwiebe. Fixes #20131.

File:

• trunk/wp-content/themes/twentytwelve/js/navigation.js (modified) (2 diffs)

trunk/wp-content/themes/twentytwelve/js/navigation.js

@@ -9 +9 @@
 
     $.fn.smallMenu = function() {
-        $( masthead ).find( '.site-navigation' ).removeClass( 'main-navigation' ).addClass( 'main-small-navigation' );
-        $( masthead ).find( '.site-navigation h3' ).removeClass( 'assistive-text' ).addClass( 'menu-toggle' );
+        masthead.find( '.site-navigation' ).removeClass( 'main-navigation' ).addClass( 'main-small-navigation' );
+        masthead.find( '.site-navigation h3' ).removeClass( 'assistive-text' ).addClass( 'menu-toggle' );
 
         $( '.menu-toggle' ).click( function() {
-            $( masthead ).find( '.menu' ).toggle();
+            masthead.find( '.menu' ).toggle();
             $( this ).toggleClass( 'toggled-on' );
         } );
@@ -33 +33 @@
                 $.fn.smallMenu();
             } else {
-                $( masthead ).find( '.site-navigation' ).removeClass( 'main-small-navigation' ).addClass( 'main-navigation' );
-                $( masthead ).find( '.site-navigation h3' ).removeClass( 'menu-toggle' ).addClass( 'assistive-text' );
-                $( masthead ).find( '.menu' ).removeAttr( 'style' );
+                masthead.find( '.site-navigation' ).removeClass( 'main-small-navigation' ).addClass( 'main-navigation' );
+                masthead.find( '.site-navigation h3' ).removeClass( 'menu-toggle' ).addClass( 'assistive-text' );
+                masthead.find( '.menu' ).removeAttr( 'style' );
             }
         }, 200 );