From 9dcb124ff7d1af62d5f6dc95f7034f3d3dca896b Mon Sep 17 00:00:00 2001
From: Kevin Sawicki <kevin@github.com>
Date: Wed, 13 Feb 2013 18:30:08 -0800
Subject: [PATCH] Move path selection to view class

---
 src/packages/command-panel/lib/path-view.coffee    | 10 ++++++++++
 src/packages/command-panel/lib/preview-list.coffee |  8 --------
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/src/packages/command-panel/lib/path-view.coffee b/src/packages/command-panel/lib/path-view.coffee
index 682236b64..930c1d9d3 100644
--- a/src/packages/command-panel/lib/path-view.coffee
+++ b/src/packages/command-panel/lib/path-view.coffee
@@ -1,6 +1,7 @@
 {View} = require 'space-pen'
 fs = require 'fs'
 OperationView = require './operation-view'
+$ = require 'jquery'
 
 module.exports =
 class PathView extends View
@@ -14,6 +15,15 @@ class PathView extends View
         for operation in operations
           @subview "operation#{operation.index}", new OperationView({operation})
 
+  initialize: ->
+    @on 'mousedown', @onPathSelected
+
+  onPathSelected: (event) =>
+    e = $(event.target)
+    e = e.parent() if e.parent().hasClass 'path'
+    if e.hasClass 'path'
+      @matches.toggle 100, => @toggleClass 'is-collapsed'
+
   expand: ->
     @matches.show()
     @removeClass 'is-collapsed'
diff --git a/src/packages/command-panel/lib/preview-list.coffee b/src/packages/command-panel/lib/preview-list.coffee
index 8e289f178..0d6b4898f 100644
--- a/src/packages/command-panel/lib/preview-list.coffee
+++ b/src/packages/command-panel/lib/preview-list.coffee
@@ -23,7 +23,6 @@ class PreviewList extends ScrollView
       @setSelectedOperationIndex(parseInt($(e.target).closest('li').data('index')))
       @executeSelectedOperation()
 
-    @on 'mousedown', 'li.path', @onPathSelected
     @command 'command-panel:collapse-all', => @collapseAllPaths()
     @command 'command-panel:expand-all', => @expandAllPaths()
     @command 'command-panel:collapse-result', @collapseSelectedPath
@@ -34,13 +33,6 @@ class PreviewList extends ScrollView
     e.children('ul.matches').hide 100, (e) ->
       $(this).closest('li.path').addClass 'is-collapsed'
 
-  onPathSelected: (event) =>
-    e = $(event.target)
-    e = e.parent() if e.parent().hasClass 'path'
-    return unless e.hasClass 'path'
-    e.children('ul.matches').toggle 100, (e) ->
-      $(this).closest('li.path').toggleClass 'is-collapsed'
-
   expandAllPaths: ->
     @children().each (index, element) -> $(element).view().expand()