From fafc1f77bdfe6a35ffb95f38793659b44038b12c Mon Sep 17 00:00:00 2001
From: Max Ogden <max@maxogden.com>
Date: Sat, 2 Jul 2011 16:08:32 -0700
Subject: [PATCH] refactor menu click event binding

---
 attachments/script/removalist.js | 38 ++++++++++++++++----------------
 1 file changed, 19 insertions(+), 19 deletions(-)

diff --git a/attachments/script/removalist.js b/attachments/script/removalist.js
index 162cbce8..6a8db21d 100644
--- a/attachments/script/removalist.js
+++ b/attachments/script/removalist.js
@@ -5,28 +5,28 @@ var removalist = function() {
   }
   
   function handleMenuClick() {
-    $( '.menu .csv' ).click(function( e ) {      
-      window.location.href = app.csvUrl;
+    $( '.menu li' ).click(function(e) {
+      if ($(e.target).hasClass('transform')) {
+        $('.dialog-overlay').show();
+        $('.dialog-container').show();
+        util.render('bulkEdit', 'dialog-content');
+        $('.cancelButton').click(function(e) {
+          $('.dialog-overlay').hide();
+          $('.dialog-container').hide();
+        })
+        $('.menu').hide();
+        $('.menu-overlay').hide();
+      }
+      
+      if ($(e.target).hasClass('csv')) {
+        window.location.href = app.csvUrl;
+      }
+      
       e.preventDefault();
-    })
-    
-    $( '.menu .transform' ).click(function( e ) {      
-      $('.dialog-overlay').show();
-      $('.dialog-container').show();
-      util.render('bulkEdit', 'dialog-content');
-      $('.cancelButton').click(function(e) {
-        $('.dialog-overlay').hide();
-        $('.dialog-container').hide();
-      })
+      
       $('.menu').hide();
       $('.menu-overlay').hide();
-      e.preventDefault();
-    })
-    
-    $( '.menu li' ).click(function() {
-      $('.menu').hide();
-      $('.menu-overlay').hide();
-    })
+    }) 
   }
   
   function renderRows(response) {