From 7de22674865522f814f762929788bfa32bfeaea6 Mon Sep 17 00:00:00 2001
From: Ben Fry <fry@processing.org>
Date: Wed, 14 May 2014 08:26:22 -0400
Subject: [PATCH] fix for #2514 to use release instead of press to fire chooser

---
 app/src/processing/mode/java/JavaEditor.java | 2 +-
 todo.txt                                     | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

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