implement active state for debugger button

benfry · benfry · commit 1f3ab674ae64 · 2015-05-19T16:56:24.000-04:00
diff --git a/java/src/processing/mode/java/JavaToolbar.java b/java/src/processing/mode/java/JavaToolbar.java
@@ -118,7 +118,10 @@ public void actionPerformed(ActionEvent e) {
         jeditor.toggleDebug();
       }
     };
-    debugButton.setReverse();
+//    debugButton.setReverse();
+    if (((JavaEditor) editor).isDebuggerEnabled()) {
+      debugButton.setSelected(true);
+    }
     box.add(debugButton);
     addGap(box);
   }
diff --git a/todo.txt b/todo.txt
@@ -1,5 +1,5 @@
 0236 (3.0a9)
-X show screen dimensions in the Preferences window
+X show screen dimensions in the Preferences window for display selector
 
 gui/pdex/debugger
 X implement the bottom half of the editor window
@@ -33,6 +33,7 @@ o   almost always used as a negative, or tied to a 'return' from fxn
 o   name isn't tied to its function, but the symptom
 X update hasJavaTabs on editor header rebuild
 X remove focus border from the Variables window
+X need active state for the butterfly
 
 in alpha 8 (but not confirmed in time)
 X "step" not working properly
@@ -43,7 +44,6 @@ X   https://github.com/processing/processing/issues/3242
 
 gui
 _ click a line number to toggle breakpoint
-_ need active state for the butterfly
 _ finish the gui
 _   https://github.com/processing/processing/issues/3072
 _ add a "what's new" window to explain features in 3

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,10 @@ public void actionPerformed(ActionEvent e) {`
`118`	`118`	`jeditor.toggleDebug();`
`119`	`119`	`}`
`120`	`120`	`};`
`121`		`- debugButton.setReverse();`
	`121`	`+// debugButton.setReverse();`
	`122`	`+ if (((JavaEditor) editor).isDebuggerEnabled()) {`
	`123`	`+ debugButton.setSelected(true);`
	`124`	`+ }`
`122`	`125`	`box.add(debugButton);`
`123`	`126`	`addGap(box);`
`124`	`127`	`}`