Add select all support to debugger

Supported in all text windows, and controlled via ctrl/cmd-A, as users expect. Fixes #399.
SoarGroup · Sep 26, 2024 · a86f3d7 · a86f3d7
1 parent 9c7c62f
commit a86f3d7
Show file tree

Hide file tree

Showing 9 changed files with 58 additions and 0 deletions.
diff --git a/Java/Debugger/src/edu/umich/soar/debugger/menu/EditMenu.java b/Java/Debugger/src/edu/umich/soar/debugger/menu/EditMenu.java
@@ -49,6 +49,14 @@ public void actionPerformed(ActionEvent e)
         }
     };
 
+    private final AbstractAction m_SelectAll = new AbstractAction("&Select All\t" + SHORTCUT_HINT + "A")
+    {
+        @Override
+        public void actionPerformed(ActionEvent e) {
+            selectAll();
+        }
+    };
+
     private final AbstractAction m_Search = new AbstractAction("&Find...\t" + SHORTCUT_HINT + "F")
     {
         @Override
@@ -76,6 +84,7 @@ private BaseMenu makeMenu(Menu parent, String title)
 
         menu.add(m_Copy);
         menu.add(m_Paste);
+        menu.add(m_SelectAll, SHORTCUT_KEY + 'A');
         menu.addSeparator();
         menu.add(m_Search, SHORTCUT_KEY + 'F');
 
@@ -117,4 +126,13 @@ private void paste()
         if (view != null)
             view.paste();
     }
+
+    private void selectAll()
+    {
+        edu.umich.soar.debugger.modules.AbstractView view = m_Frame
+            .getMainWindow().getFocusView();
+        if (view != null) {
+            view.selectAll();
+        }
+    }
 }
diff --git a/Java/Debugger/src/edu/umich/soar/debugger/modules/AbstractFixedView.java b/Java/Debugger/src/edu/umich/soar/debugger/modules/AbstractFixedView.java
@@ -73,6 +73,11 @@ public void copy()
         // Usually nothing to copy in fixed views
     }
 
+    @Override
+    public void selectAll() {
+        // Usually nothing to select in fixed views
+    }
+
     /************************************************************************
      *
      * Search for the next occurance of 'text' in this view and place the

diff --git a/Java/Debugger/src/edu/umich/soar/debugger/modules/AbstractTextView.java b/Java/Debugger/src/edu/umich/soar/debugger/modules/AbstractTextView.java
@@ -83,6 +83,12 @@ public void copy()
         m_Text.copy();
     }
 
+    @Override
+    public void selectAll()
+    {
+        m_Text.selectAll();
+    }
+
     /************************************************************************
      *
      * Search for the next occurrence of {@code text} in this view and place the

diff --git a/Java/Debugger/src/edu/umich/soar/debugger/modules/AbstractView.java b/Java/Debugger/src/edu/umich/soar/debugger/modules/AbstractView.java
@@ -201,6 +201,8 @@ public void paste()
             this.executeAgentCommand(textData, true);
     }
 
+    public abstract void selectAll();
+
     // Creates a convenient listener for Control-V.
     // Adding a key listener for this event allows us to call our paste method
     // so commands are pasted into the command

diff --git a/Java/Debugger/src/edu/umich/soar/debugger/modules/EditorView.java b/Java/Debugger/src/edu/umich/soar/debugger/modules/EditorView.java
@@ -114,6 +114,11 @@ public void paste()
         m_Text.paste();
     }
 
+    @Override
+    public void selectAll() {
+        m_Text.selectAll();
+    }
+
     @Override
     public void setTextFont(Font f)
     {

diff --git a/Java/Debugger/src/edu/umich/soar/debugger/modules/FoldingTextView.java b/Java/Debugger/src/edu/umich/soar/debugger/modules/FoldingTextView.java
@@ -613,6 +613,12 @@ public void copy()
         m_FoldingText.getTextWindow().copy();
     }
 
+    @Override
+    public void selectAll()
+    {
+        m_FoldingText.getTextWindow().selectAll();
+    }
+
     /************************************************************************
      *
      * Search for the next occurance of 'text' in this view and place the

diff --git a/Java/Debugger/src/edu/umich/soar/debugger/modules/RHSBarChartView.java b/Java/Debugger/src/edu/umich/soar/debugger/modules/RHSBarChartView.java
@@ -241,6 +241,11 @@ public void copy()
     {
     }
 
+    @Override
+    public void selectAll()
+    {
+    }
+
     @Override
     public void displayText(String text)
     {

diff --git a/Java/Debugger/src/edu/umich/soar/debugger/modules/RHSFunTextView.java b/Java/Debugger/src/edu/umich/soar/debugger/modules/RHSFunTextView.java
@@ -260,6 +260,11 @@ public void copy()
     {
         textBox.copy();
     }
+    @Override
+    public void selectAll()
+    {
+        textBox.selectAll();
+    }
 
     @Override
     protected ParseSelectedText.SelectedObject getCurrentSelection(int mouseX,

diff --git a/Java/Debugger/src/edu/umich/soar/debugger/modules/TextTraceView.java b/Java/Debugger/src/edu/umich/soar/debugger/modules/TextTraceView.java
@@ -81,6 +81,12 @@ public void copy()
         m_Text.copy();
     }
 
+    @Override
+    public void selectAll()
+    {
+        m_Text.selectAll();
+    }
+
     /********************************************************************************************
      *
      * Scroll the display control to the bottom