From 851ba7cecf30b3dcc8df09d2999a6fdd73ebe29d Mon Sep 17 00:00:00 2001
From: Kevin Sweeney <kevinsweeney@gmail.com>
Date: Fri, 4 Oct 2013 01:19:03 -0400
Subject: [PATCH] Store DOM selectors where safe

More optimizations could still be done, but without being able to see where the function calls are being made, I assumed a worst-case scenario which could require fresh selectors for content that was loaded via XHR.
---
 js/theme.js | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/js/theme.js b/js/theme.js
index 38da2e8..a7c0843 100644
--- a/js/theme.js
+++ b/js/theme.js
@@ -95,10 +95,11 @@
     var closeText = 'Close Menu';
     $('a.handle').html(openText);
     function menuDropdown(){
-      $('a.handle').click(function(){
+      var handle = $('a.handle');
+      handle.click(function(){
         $('.toggle-nav').slideToggle('fast');
       });
-      $('a.handle').toggle(function (){
+      handle.toggle(function (){
         $(this).text(closeText).stop();
         $(this).addClass('handle-close');
         }, function(){
@@ -108,8 +109,9 @@
     }
 
     function destroyDropdown(){
-      $('a.handle').unbind('click');
-      $('a.handle').unbind('toggle');
+      var handle = $('a.handle');
+      handle.unbind('click');
+      handle.unbind('toggle');
     }
     
     function destroySecondary(){
@@ -118,8 +120,9 @@
 
     //Seconday Links Dropdown
     function secondaryDropdown(){
-      $('#secondary-menu h2').click(function(){
-        $('#secondary-menu ul').slideToggle('fast');
+      var secondary_menu = $('#secondary-menu');
+      $('h2', secondary_menu).click(function(){
+        $('ul', secondary_menu).slideToggle('fast');
         $(this).toggleClass('arrow-up').toggleClass('arrow-down');
       });
     }
@@ -132,4 +135,4 @@
       $('.share-box').remove().insertAfter('.change-box');
     }
 
-})(jQuery);
\ No newline at end of file
+})(jQuery);