moving more things into Editor

benfry · benfry · commit 68bfec01c337 · 2016-08-06T14:52:42.000-04:00
diff --git a/app/src/processing/app/ui/Editor.java b/app/src/processing/app/ui/Editor.java
@@ -278,6 +278,11 @@ public void actionPerformed(ActionEvent e) {
         }
       });
     }
+    textarea.addCaretListener(new CaretListener() {
+      public void caretUpdate(CaretEvent e) {
+        updateEditorStatus();
+      }
+    });
 
     footer = createFooter();
 
diff --git a/core/todo.txt b/core/todo.txt
@@ -12,10 +12,15 @@ X Fix resizing targets for async save
 X   https://github.com/processing/processing/pull/4607
 X   https://github.com/processing/processing/issues/4578
 
+contrib
+X Make loadStrings() and loadJSONObject/loadJSONArray() error msgs consistent
+X   https://github.com/processing/processing/issues/4265
+X   https://github.com/processing/processing/pull/4268
+
+
 _ disable OpenGL ES on Linux? 
 _   https://github.com/processing/processing/issues/4584
 
-
 _ Can't render PGraphics object using image() within a PDF
 _   https://github.com/processing/processing/issues/4473
 
diff --git a/java/src/processing/mode/java/JavaEditor.java b/java/src/processing/mode/java/JavaEditor.java
@@ -80,21 +80,6 @@ protected JavaEditor(Base base, String path, EditorState state,
     debugger = new Debugger(this);
     inspector = new VariableInspector(this);
 
-    // set action on frame close
-    //        addWindowListener(new WindowAdapter() {
-    //            @Override
-    //            public void windowClosing(WindowEvent e) {
-    //                onWindowClosing(e);
-    //            }
-    //        });
-
-
-//    // load settings from theme.txt
-//    breakpointColor = mode.getColor("breakpoint.bgcolor");
-//    breakpointMarkerColor = mode.getColor("breakpoint.marker.color");
-//    currentLineColor = mode.getColor("currentline.bgcolor");
-//    currentLineMarkerColor = mode.getColor("currentline.marker.color");
-
     // set breakpoints from marker comments
     for (LineID lineID : stripBreakpointComments()) {
       //System.out.println("setting: " + lineID);
@@ -119,7 +104,7 @@ protected JavaEditor(Base base, String path, EditorState state,
     // add our hacked version back to the editor
     box.add(textAndError);
 
-    getJavaTextArea().setMode(jmode);
+    getPdeTextArea().setMode(jmode);
 
     preprocessingService = new PreprocessingService(this);
     pdex = new PDEX(this, preprocessingService);
@@ -134,12 +119,6 @@ public void windowLostFocus(WindowEvent e) {
 
       public void windowGainedFocus(WindowEvent e) { }
     });
-
-    textarea.addCaretListener(new CaretListener() {
-      public void caretUpdate(CaretEvent e) {
-        updateEditorStatus();
-      }
-    });
   }
 
 
diff --git a/java/src/processing/mode/java/pdex/JavaTextAreaPainter.java b/java/src/processing/mode/java/pdex/JavaTextAreaPainter.java
@@ -234,7 +234,7 @@ protected void updateTweakInterface(List<List<Handle>> handles,
 		this.handles = handles;
 		this.colorBoxes = colorBoxes;
 
-		initTweakInterfacePositions();
+		updateTweakInterfacePositions();
 		repaint();
 	}
 
@@ -244,7 +244,7 @@ protected void updateTweakInterface(List<List<Handle>> handles,
 	* synchronize this method to prevent the execution of 'paint' in the middle.
 	* (don't paint while we make changes to the text of the editor)
 	*/
-	private synchronized void initTweakInterfacePositions() {
+	private synchronized void updateTweakInterfacePositions() {
 		SketchCode[] code = getEditor().getSketch().getCode();
 		int prevScroll = textArea.getVerticalScrollPosition();
 		String prevText = textArea.getText();
diff --git a/todo.txt b/todo.txt
@@ -51,6 +51,8 @@ _   https://github.com/processing/processing/issues/3965
 medium
 _ Move general PDE code out of JavaMode and into general base classes
 _   https://github.com/processing/processing/issues/4606
+_ detect changes in case with libraries
+_   https://github.com/processing/processing/issues/4507
 
 lower
 _ make when opening new editor window, open on the same display as current

Original file line number	Diff line number	Diff line change
`@@ -278,6 +278,11 @@ public void actionPerformed(ActionEvent e) {`
`278`	`278`	`}`
`279`	`279`	`});`
`280`	`280`	`}`
	`281`	`+ textarea.addCaretListener(new CaretListener() {`
	`282`	`+ public void caretUpdate(CaretEvent e) {`
	`283`	`+ updateEditorStatus();`
	`284`	`+ }`
	`285`	`+ });`
`281`	`286`
`282`	`287`	`footer = createFooter();`
`283`	`288`