onColorChangedListener triggers way to often (and first values are not valid)

[larswerkman]

onColorChangedListener triggers way to often (and first values are not valid)

[kj800x]

This is a major issue that I'm also experiencing.

[kj800x]

I've looked into the suggestion buttairfly gave, but that just delegates the event triggering to the bars (0 bars = 0 events, 1 bar = 1 event, 2 bars = 2 events). A better way to approach the problem would be for the colorpicker to ask the bars for their values, and use that to calculate the new color, and then only the colorpicker should be allowed to trigger the event. I'm working on a pull request for this.

[jlarysz]

I can't get this color picker to work in Android IDE 1.5. I'm stuck on a duplicate class in the build:

Execution failed for task ':app:transformClassesWithJarMergingForDebug'.

com.android.build.api.transform.TransformException: java.util.zip.ZipException: duplicate entry: com/larswerkman/holocolorpicker/ColorPicker$OnColorChangedListener.class

and I can't find it. I've been looking all day - I think I have to try another color picker.

[holgi-s]

Using the 1.5 version I also noticd that when using a SVBar the onColorChanged is triggered twice, and the 1st event actually contains the wrong color.

In ColorPicker.onTouchEvent()
1st event on line 631:
setNewCenterColor(mCenterNewColor = calculateColor(mAngle));
with the color of the circle, not respecting saturation and value

2nd event on line 646:
mSVbar.setColor(mColor);
this time with the correct color