diff --git a/app/src/processing/mode/java/JavaEditor.java b/app/src/processing/mode/java/JavaEditor.java
index 681bde70f..70e083ea4 100644
--- a/app/src/processing/mode/java/JavaEditor.java
+++ b/app/src/processing/mode/java/JavaEditor.java
@@ -616,7 +616,7 @@ public class JavaEditor extends Editor {
       setMaximumSize(new Dimension(30, 20));
       
       addMouseListener(new MouseAdapter() {
-        public void mousePressed(MouseEvent e) {
+        public void mouseReleased(MouseEvent e) {
           Color color = Preferences.getColor(prefName);
           chooser = new ColorChooser(JavaEditor.this, true, color, "Select", ColorPreference.this);
           chooser.show();
diff --git a/todo.txt b/todo.txt
index 972114d51..c69083e14 100644
--- a/todo.txt
+++ b/todo.txt
@@ -1,4 +1,7 @@
 0227 pde
+X use mouseReleased() instead of mousePressed() in color selector
+X   otherwise it registers the release as a click in the color window
+X   https://github.com/processing/processing/issues/2514
 
 
 medium