From ba8cf7742af882f563e87c08056da79bdfabc880 Mon Sep 17 00:00:00 2001
From: Tyler Renelle <tylerrenelle@gmail.com>
Date: Fri, 1 Mar 2013 18:02:53 -0500
Subject: [PATCH] sortable cleanup

---
 src/app/browser.coffee | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/app/browser.coffee b/src/app/browser.coffee
index 2830fefd48..7cfdb67d59 100644
--- a/src/app/browser.coffee
+++ b/src/app/browser.coffee
@@ -81,10 +81,7 @@ loadJavaScripts = (model) ->
 ###
 setupSortable = (model) ->
   unless (model.get('_view.mobileDevice') == true) #don't do sortable on mobile
-    # Make the lists draggable using jQuery UI
-    # Note, have to setup helper function here and call it for each type later
-    # due to variable binding of "type"
-    setupSortable = (type) ->
+    _.each ['habit', 'daily', 'todo', 'reward'], (type) ->
       $("ul.#{type}s").sortable
         dropOnEmpty: false
         cursor: "move"
@@ -102,7 +99,6 @@ setupSortable = (model) ->
           # Also, note that refList index arguments can either be an index
           # or the item's id property
           model.at("_#{type}List").pass(ignore: domId).move {id}, to
-    _.each ['habit', 'daily', 'todo', 'reward'], (type) -> setupSortable(type)
 
 setupTooltips = (model) ->
   $('[rel=tooltip]').tooltip()