From 9caf096564da324591da5c744d8bf0df5e0c0d57 Mon Sep 17 00:00:00 2001
From: Manindra Moharana <mkmoharana29@gmail.com>
Date: Tue, 15 Jul 2014 18:38:14 +0530
Subject: [PATCH] refactoring, moving things around

---
 .../mode/experimental/SketchOutline.java      | 21 ++++++++++++-------
 .../mode/experimental/TabOutline.java         |  4 ++--
 2 files changed, 15 insertions(+), 10 deletions(-)

diff --git a/pdex/src/processing/mode/experimental/SketchOutline.java b/pdex/src/processing/mode/experimental/SketchOutline.java
index c95ef073e..3df7c985b 100644
--- a/pdex/src/processing/mode/experimental/SketchOutline.java
+++ b/pdex/src/processing/mode/experimental/SketchOutline.java
@@ -52,10 +52,21 @@ public class SketchOutline {
   protected JTextField searchField;
 
   protected DebugEditor editor;
+  
+  protected boolean internalSelection = false;
 
   public SketchOutline(DefaultMutableTreeNode codeTree, ErrorCheckerService ecs) {
     errorCheckerService = ecs;
     editor = ecs.getEditor();
+    soNode = new DefaultMutableTreeNode();
+    generateSketchOutlineTree(soNode, codeTree);
+    soNode = (DefaultMutableTreeNode) soNode.getChildAt(0);
+    tempNode = soNode;
+    soTree = new JTree(soNode);
+    createGUI();
+  }
+  
+  private void createGUI(){
     frmOutlineView = new JFrame();
     frmOutlineView.setAlwaysOnTop(true);
     frmOutlineView.setUndecorated(true);
@@ -79,11 +90,7 @@ public class SketchOutline {
     panelTop.add(searchField);
 
     jsp = new JScrollPane();
-    soNode = new DefaultMutableTreeNode();
-    generateSketchOutlineTree(soNode, codeTree);
-    soNode = (DefaultMutableTreeNode) soNode.getChildAt(0);
-    tempNode = soNode;
-    soTree = new JTree(soNode);
+    
     soTree.getSelectionModel()
         .setSelectionMode(TreeSelectionModel.SINGLE_TREE_SELECTION);
     soTree.setRootVisible(false);
@@ -117,11 +124,8 @@ public class SketchOutline {
                                    + (editor.ta.getHeight() - frmOutlineView
                                        .getHeight()) / 2);
     addListeners();
-
   }
 
-  protected boolean internalSelection = false;
-
   protected void addListeners() {
 
     searchField.addKeyListener(new KeyAdapter() {
@@ -257,6 +261,7 @@ public class SketchOutline {
               awrap.highlightNode(errorCheckerService.astGenerator);
               // log(awrap);
               //errorCheckerService.highlightNode(awrap);
+              close();
             }
           }
         };
diff --git a/pdex/src/processing/mode/experimental/TabOutline.java b/pdex/src/processing/mode/experimental/TabOutline.java
index 454041b8a..3059877de 100644
--- a/pdex/src/processing/mode/experimental/TabOutline.java
+++ b/pdex/src/processing/mode/experimental/TabOutline.java
@@ -50,10 +50,10 @@ public class TabOutline {
   public TabOutline(ErrorCheckerService ecs) {
     errorCheckerService = ecs;
     editor = ecs.getEditor();
-    createAndShowGUI();
+    createGUI();
   }
 
-  private void createAndShowGUI() {
+  private void createGUI() {
     frmOutlineView = new JFrame();
     frmOutlineView.setAlwaysOnTop(true);
     frmOutlineView.setUndecorated(true);