kittkattNumber1 / android-color-picker

Automatically exported from code.google.com/p/android-color-picker
0 stars 1 forks source link

Does not work on tablets (Android 3.0 Honeycomb) because what color I select is not the same as the one shown and returned #4

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I'm using Acer Iconia Tab A500 and Android 3.0.1

The color I'm selecting with the finger in the big gradient selector rectangle 
is not shown correctly in the second small result rectangle or return 
correctly. I get a darker color.

I'm using something like:

int currtentColor = 0xffffffff;
AmbilWarnaDialog dialog = new AmbilWarnaDialog(YourClassName.this, 
currtentColor, new OnAmbilWarnaListener()
{
@Override
public void onCancel(AmbilWarnaDialog dialog) {
}
@Override
public void onOk(AmbilWarnaDialog dialog, int color) {
}
});
dialog.show();

Original issue reported on code.google.com by nerva...@gmail.com on 27 May 2011 at 3:17

GoogleCodeExporter commented 9 years ago
Thanks for the report, please provide a screenshot.

Original comment by yukuku on 30 May 2011 at 12:24

GoogleCodeExporter commented 9 years ago
I think the OP is observing a bug where the hardware-accelerated renderer fails 
to shade the big color box. I found that my Thinkpad Tablet, too: With hardware 
acceleration enabled, the big gradient selector is only white-to-color in the 
left-to-right direction, but of constant color in the vertical direction. With 
hw acceleration disabled, it is additionally shaded in the vertical direction, 
the bottom being all black. I added the following workaround on my tablet:

viewSatVal.setLayerType(View.LAYER_TYPE_SOFTWARE, null);

Original comment by vbraun.n...@gmail.com on 12 Oct 2011 at 1:36

GoogleCodeExporter commented 9 years ago

Original comment by yukuku on 13 Oct 2011 at 1:28

GoogleCodeExporter commented 9 years ago
Thanks, it's fixed in r3.

Original comment by yukuku on 3 Nov 2011 at 8:00

GoogleCodeExporter commented 9 years ago
Thanks! In landscape it works perfectly now, but in the portrait layout xml the 
wrong view is set to software render. The attached trivial patch fixes it.

Original comment by vbraun.n...@gmail.com on 8 Dec 2011 at 1:28

Attachments:

GoogleCodeExporter commented 9 years ago
Hi vbraun.name,

Thanks for the patch. I didn't realize it. It's fixed in r4.

Original comment by yukuku on 9 Dec 2011 at 7:04