add workaround for setTitle() resetting cursor() (fixes processing#3472)

benfry · benfry · commit 4b860a2c1dfe · 2015-08-08T14:54:17.000-04:00
diff --git a/core/src/processing/awt/PSurfaceAWT.java b/core/src/processing/awt/PSurfaceAWT.java
@@ -552,6 +552,12 @@ public Object getNative() {
   @Override
   public void setTitle(String title) {
     frame.setTitle(title);
+    // Workaround for apparent Java bug?
+    // https://github.com/processing/processing/issues/3472
+    if (cursorVisible) {
+      hideCursor();
+      showCursor();
+    }
   }
 
 
diff --git a/core/src/processing/opengl/PSurfaceJOGL.java b/core/src/processing/opengl/PSurfaceJOGL.java
@@ -975,11 +975,12 @@ private static char hackToChar(short code, char def) {
     return def;
   }
 
+
   // . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . .
 
 
   public void setCursor(int kind) {
-    System.err.println("Cursor types not supported in OpenGL, provide your cursor image");
+    PGraphics.showWarning("Cursor types not yet supported in OpenGL, provide your cursor image");
   }
 
 
diff --git a/core/todo.txt b/core/todo.txt
@@ -4,6 +4,8 @@ X   prevents "fullScreen() cannot be used here" message on startup
 X   https://github.com/processing/processing/issues/3545
 X throw an error when using methods that require sketchPath outside setup()
 X   https://github.com/processing/processing/issues/3433
+X cursor(CROSS) breaks when using surface.setTitle()
+X   https://github.com/processing/processing/issues/3472
 
 fixed earlier
 X blend() and copy() are not pixel accurate for copy/scale
@@ -80,6 +82,8 @@ opengl
 _ hard crash at 1920x1080, mirrored, Casey's GT 650M 1GB
 _ window loses focus after maximizing
 _   https://github.com/processing/processing/issues/3339
+_ Implement standard cursor types in OpenGL
+_   https://github.com/processing/processing/issues/3554
 
 
 opengl questions
diff --git a/todo.txt b/todo.txt
@@ -47,6 +47,8 @@ _   probably have to add the script/Processing.app location to user's path
 gui
 _ fix red in sidebar, the squiggly line beneath code
 _ show hover text with 'debug'
+_ show number of updates available in the footer
+_   https://github.com/processing/processing/issues/3518
 _ don't show breakpoints when debugger is off
 _   https://github.com/processing/processing/issues/3093
 _ no setting breakpoints when debugger is off

Original file line number	Diff line number	Diff line change
`@@ -975,11 +975,12 @@ private static char hackToChar(short code, char def) {`
`975`	`975`	`return def;`
`976`	`976`	`}`
`977`	`977`
	`978`	`+`
`978`	`979`	`// . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . .`
`979`	`980`
`980`	`981`
`981`	`982`	`public void setCursor(int kind) {`
`982`		`- System.err.println("Cursor types not supported in OpenGL, provide your cursor image");`
	`983`	`+ PGraphics.showWarning("Cursor types not yet supported in OpenGL, provide your cursor image");`
`983`	`984`	`}`
`984`	`985`
`985`	`986`