ColorPickerView onMeasure does not account for constrained by height about colorpicker HOT 6 CLOSED

Sorry, I do not quite understand the issue, what the expected layout?

from colorpicker.

ColorPickerView .onMeasure override does not account for ColorPickerView being constrained by height rather than by width. In my use-case I am constraining the ColorPickerView by height and have it wrap_content for width, being placed in a constraint layout with other views above and below it.

android:layout_width="wrap_content" android:layout_height="0dp"

In which case onMeasure fails to compute the desired width and height correctly and the ColorPickerView does not appear on the screen at all.

from colorpicker.

Hi, I added a layout for landscape to the sample app using:

android:layout_width="wrap_content"
android:layout_height="0dp"

And the measure seems work properly:

from colorpicker.

You don't have anything on top or below the ColorPickerView. What happens if you place a view with height of 96dp to the top of the color view and another view 120dp of height below the ColorPickerView?
I fail to see how this will work considering there is no code in ColorPickerView.onMeasure to handle being constrained by height.

from colorpicker.

Tested to layout with top and bottom views and didn't see the issue:

from colorpicker.

Looks fine in the designer. Using it in an alert dialog with custom view is not working for me as the alert dialog by default uses wrap content for height (causing the color picker not to show up at all), but that's apparently a different issue.

from colorpicker.