1
0
mirror of https://github.com/lvgl/lvgl.git synced 2025-01-14 06:42:58 +08:00

Consolidate separate h, s, & v vars in to single lv_color_hsv_t var

This commit is contained in:
Paul Peavyhouse 2019-09-28 01:41:26 -07:00
parent 1a5001d7a3
commit 1a3d61e074
2 changed files with 105 additions and 120 deletions

View File

@ -9,6 +9,7 @@
#include "lv_cpicker.h"
#if LV_USE_CPICKER != 0
#include "../lv_core/lv_debug.h"
#include "../lv_draw/lv_draw_arc.h"
#include "../lv_themes/lv_theme.h"
#include "../lv_core/lv_indev.h"
@ -34,6 +35,10 @@
#define LV_CPICKER_DEF_VALUE 100
#endif
#ifndef LV_CPICKER_DEF_HSV
#define LV_CPICKER_DEF_HSV ((lv_color_hsv_t){LV_CPICKER_DEF_HUE, LV_CPICKER_DEF_SATURATION, LV_CPICKER_DEF_VALUE})
#endif
#ifndef LV_CPICKER_DEF_INDICATOR_TYPE
#define LV_CPICKER_DEF_INDICATOR_TYPE LV_CPICKER_INDICATOR_CIRCLE
#endif
@ -92,7 +97,7 @@ lv_obj_t * lv_cpicker_create(lv_obj_t * par, const lv_obj_t * copy)
LV_LOG_TRACE("color_picker create started");
lv_obj_t * new_cpicker = lv_obj_create(par, copy);
lv_mem_assert(new_cpicker);
LV_ASSERT_MEM(new_cpicker);
if(new_cpicker == NULL) return NULL;
if(ancestor_signal == NULL) ancestor_signal = lv_obj_get_signal_cb(new_cpicker);
@ -100,14 +105,12 @@ lv_obj_t * lv_cpicker_create(lv_obj_t * par, const lv_obj_t * copy)
/*Allocate the extended data*/
lv_cpicker_ext_t * ext = lv_obj_allocate_ext_attr(new_cpicker, sizeof(lv_cpicker_ext_t));
lv_mem_assert(ext);
LV_ASSERT_MEM(ext);
if(ext == NULL) return NULL;
/*Initialize the allocated 'ext' */
ext->hue = LV_CPICKER_DEF_HUE;
ext->prev_hue = ext->hue;
ext->saturation = LV_CPICKER_DEF_SATURATION;
ext->value = LV_CPICKER_DEF_VALUE;
ext->hsv = LV_CPICKER_DEF_HSV;
ext->prev_hsv = ext->hsv;
ext->indicator.style = &lv_style_plain;
ext->indicator.type = LV_CPICKER_DEF_INDICATOR_TYPE;
ext->color_mode = LV_CPICKER_COLOR_MODE_HUE;
@ -201,7 +204,7 @@ void lv_cpicker_set_hue(lv_obj_t * cpicker, uint16_t hue)
{
lv_cpicker_ext_t * ext = lv_obj_get_ext_attr(cpicker);
ext->hue = hue % 360;
ext->hsv.h = hue % 360;
lv_obj_invalidate(cpicker);
}
@ -215,7 +218,7 @@ void lv_cpicker_set_saturation(lv_obj_t * cpicker, uint8_t saturation)
{
lv_cpicker_ext_t * ext = lv_obj_get_ext_attr(cpicker);
ext->saturation = saturation % 100;
ext->hsv.s = saturation % 100;
lv_obj_invalidate(cpicker);
}
@ -229,7 +232,21 @@ void lv_cpicker_set_value(lv_obj_t * cpicker, uint8_t val)
{
lv_cpicker_ext_t * ext = lv_obj_get_ext_attr(cpicker);
ext->value = val % 100;
ext->hsv.v = val % 100;
lv_obj_invalidate(cpicker);
}
/**
* Set the current hsv of a colorpicker.
* @param cpicker pointer to colorpicker object
* @param color current selected hsv
*/
void lv_cpicker_set_hsv(lv_obj_t * cpicker, lv_color_hsv_t hsv)
{
lv_cpicker_ext_t * ext = lv_obj_get_ext_attr(cpicker);
ext->hsv = hsv;
lv_obj_invalidate(cpicker);
}
@ -241,12 +258,7 @@ void lv_cpicker_set_value(lv_obj_t * cpicker, uint8_t val)
*/
void lv_cpicker_set_color(lv_obj_t * cpicker, lv_color_t color)
{
lv_cpicker_ext_t * ext = lv_obj_get_ext_attr(cpicker);
lv_color_hsv_t hsv = lv_color_rgb_to_hsv(color.ch.red, color.ch.green, color.ch.blue);
ext->hue = hsv.h;
lv_obj_invalidate(cpicker);
lv_cpicker_set_hsv(cpicker, lv_color_rgb_to_hsv(color.ch.red, color.ch.green, color.ch.blue));
}
/**
@ -325,7 +337,7 @@ lv_style_t * lv_cpicker_get_style(const lv_obj_t * cpicker, lv_cpicker_style_t t
}
/**
* Get the current hue of a colorpicker.
* Get the current selected hue of a colorpicker.
* @param cpicker pointer to colorpicker object
* @return hue current selected hue
*/
@ -333,11 +345,11 @@ uint16_t lv_cpicker_get_hue(lv_obj_t * cpicker)
{
lv_cpicker_ext_t * ext = lv_obj_get_ext_attr(cpicker);
return ext->hue;
return ext->hsv.h;
}
/**
* Get the current saturation of a colorpicker.
* Get the current selected saturation of a colorpicker.
* @param cpicker pointer to colorpicker object
* @return current selected saturation
*/
@ -345,11 +357,11 @@ uint8_t lv_cpicker_get_saturation(lv_obj_t * cpicker)
{
lv_cpicker_ext_t * ext = lv_obj_get_ext_attr(cpicker);
return ext->saturation;
return ext->hsv.s;
}
/**
* Get the current hue of a colorpicker.
* Get the current selected hue of a colorpicker.
* @param cpicker pointer to colorpicker object
* @return current selected value
*/
@ -357,7 +369,19 @@ uint8_t lv_cpicker_get_value(lv_obj_t * cpicker)
{
lv_cpicker_ext_t * ext = lv_obj_get_ext_attr(cpicker);
return ext->value;
return ext->hsv.v;
}
/**
* Get the current selected hsv of a colorpicker.
* @param cpicker pointer to colorpicker object
* @return current selected hsv
*/
lv_color_hsv_t lv_cpicker_get_hsv(lv_obj_t * cpicker)
{
lv_cpicker_ext_t * ext = lv_obj_get_ext_attr(cpicker);
return ext->hsv;
}
/**
@ -369,7 +393,7 @@ lv_color_t lv_cpicker_get_color(lv_obj_t * cpicker)
{
lv_cpicker_ext_t * ext = lv_obj_get_ext_attr(cpicker);
return lv_color_hsv_to_rgb(ext->hue, ext->saturation, ext->value);
return lv_color_hsv_to_rgb(ext->hsv.h, ext->hsv.s, ext->hsv.v);
}
/*=====================
@ -452,13 +476,13 @@ static lv_color_t angle_to_mode_color(lv_cpicker_ext_t * ext, uint16_t angle)
{
default:
case LV_CPICKER_COLOR_MODE_HUE:
color = lv_color_hsv_to_rgb(angle % 360, ext->saturation, ext->value);
color = lv_color_hsv_to_rgb(angle % 360, ext->hsv.s, ext->hsv.v);
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
color = lv_color_hsv_to_rgb(ext->hue, (angle % 360) / 360.0 * 100.0, ext->value);
color = lv_color_hsv_to_rgb(ext->hsv.h, (angle % 360) / 360.0 * 100.0, ext->hsv.v);
break;
case LV_CPICKER_COLOR_MODE_VALUE:
color = lv_color_hsv_to_rgb(ext->hue, ext->saturation, (angle % 360) / 360.0 * 100.0);
color = lv_color_hsv_to_rgb(ext->hsv.h, ext->hsv.s, (angle % 360) / 360.0 * 100.0);
break;
}
return color;
@ -471,13 +495,13 @@ static uint16_t mode_color_to_angle(lv_cpicker_ext_t * ext)
{
default:
case LV_CPICKER_COLOR_MODE_HUE:
angle = ext->hue;
angle = ext->hsv.h;
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
angle = ext->saturation / 100.0 * 360.0;
angle = ext->hsv.s / 100.0 * 360.0;
break;
case LV_CPICKER_COLOR_MODE_VALUE:
angle = ext->value / 100.0 * 360.0;
angle = ext->hsv.v / 100.0 * 360.0;
break;
}
return angle;
@ -490,13 +514,13 @@ static lv_coord_t lv_cpicker_get_indicator_coord(lv_style_t * style, lv_cpicker_
{
default:
case LV_CPICKER_COLOR_MODE_HUE:
ind_pos += ext->hue / 360.0 * ext->rect_gradient_w;
ind_pos += ext->hsv.h / 360.0 * ext->rect_gradient_w;
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
ind_pos += ext->saturation / 100.0 * ext->rect_gradient_w;
ind_pos += ext->hsv.s / 100.0 * ext->rect_gradient_w;
break;
case LV_CPICKER_COLOR_MODE_VALUE:
ind_pos += ext->value / 100.0 * ext->rect_gradient_w;
ind_pos += ext->hsv.v / 100.0 * ext->rect_gradient_w;
break;
}
return ind_pos;
@ -610,7 +634,7 @@ static void lv_cpicker_draw_disc_indicator(lv_area_t * mask, lv_style_t * style,
lv_draw_rect(&center_area, mask, &styleCenterBackground, opa_scale);
/*draw the center color indicator*/
style->body.main_color = lv_color_hsv_to_rgb(ext->hue, ext->saturation, ext->value);
style->body.main_color = lv_color_hsv_to_rgb(ext->hsv.h, ext->hsv.s, ext->hsv.v);
style->body.grad_color = style->body.main_color;
style->body.radius = LV_RADIUS_CIRCLE;
lv_draw_rect(&center_ind_area, mask, style, opa_scale);
@ -1181,13 +1205,13 @@ static lv_res_t lv_cpicker_signal(lv_obj_t * cpicker, lv_signal_t sign, void * p
switch(ext->color_mode)
{
case LV_CPICKER_COLOR_MODE_HUE:
ext->hue = (ext->hue + 1) % 360;
ext->hsv.h = (ext->hsv.h + 1) % 360;
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
ext->saturation = (ext->saturation + 1) % 100;
ext->hsv.s = (ext->hsv.s + 1) % 100;
break;
case LV_CPICKER_COLOR_MODE_VALUE:
ext->value = (ext->value + 1) % 100;
ext->hsv.v = (ext->hsv.v + 1) % 100;
break;
}
@ -1201,13 +1225,13 @@ static lv_res_t lv_cpicker_signal(lv_obj_t * cpicker, lv_signal_t sign, void * p
switch(ext->color_mode)
{
case LV_CPICKER_COLOR_MODE_HUE:
ext->hue = ext->hue > 0?(ext->hue - 1):360;
ext->hsv.h = ext->hsv.h > 0?(ext->hsv.h - 1):360;
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
ext->saturation = ext->saturation > 0?(ext->saturation - 1):100;
ext->hsv.s = ext->hsv.s > 0?(ext->hsv.s - 1):100;
break;
case LV_CPICKER_COLOR_MODE_VALUE:
ext->value = ext->value > 0?(ext->value - 1):100;
ext->hsv.v = ext->hsv.v > 0?(ext->hsv.v - 1):100;
break;
}
@ -1221,13 +1245,13 @@ static lv_res_t lv_cpicker_signal(lv_obj_t * cpicker, lv_signal_t sign, void * p
switch(ext->color_mode)
{
case LV_CPICKER_COLOR_MODE_HUE:
ext->hue = (ext->hue + 1) % 360;
ext->hsv.h = (ext->hsv.h + 1) % 360;
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
ext->saturation = (ext->saturation + 1) % 100;
ext->hsv.s = (ext->hsv.s + 1) % 100;
break;
case LV_CPICKER_COLOR_MODE_VALUE:
ext->value = (ext->value + 1) % 100;
ext->hsv.v = (ext->hsv.v + 1) % 100;
break;
}
@ -1241,13 +1265,13 @@ static lv_res_t lv_cpicker_signal(lv_obj_t * cpicker, lv_signal_t sign, void * p
switch(ext->color_mode)
{
case LV_CPICKER_COLOR_MODE_HUE:
ext->hue = ext->hue > 0?(ext->hue - 1):360;
ext->hsv.h = ext->hsv.h > 0?(ext->hsv.h - 1):360;
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
ext->saturation = ext->saturation > 0?(ext->saturation - 1):100;
ext->hsv.s = ext->hsv.s > 0?(ext->hsv.s - 1):100;
break;
case LV_CPICKER_COLOR_MODE_VALUE:
ext->value = ext->value > 0?(ext->value - 1):100;
ext->hsv.v = ext->hsv.v > 0?(ext->hsv.v - 1):100;
break;
}
@ -1279,40 +1303,6 @@ static lv_res_t lv_cpicker_signal(lv_obj_t * cpicker, lv_signal_t sign, void * p
return res;
}
static void lv_cpicker_prev_hsv_save(lv_cpicker_ext_t * ext)
{
switch(ext->color_mode)
{
case LV_CPICKER_COLOR_MODE_HUE:
ext->prev_hue = ext->hue;
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
ext->prev_saturation = ext->saturation;
break;
case LV_CPICKER_COLOR_MODE_VALUE:
ext->prev_value = ext->value;
break;
}
}
static void lv_cpicker_prev_hsv_restore(lv_obj_t * cpicker,
lv_cpicker_ext_t * ext)
{
switch(ext->color_mode)
{
case LV_CPICKER_COLOR_MODE_HUE:
ext->prev_hue = ext->hue;
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
ext->prev_saturation = ext->saturation;
break;
case LV_CPICKER_COLOR_MODE_VALUE:
ext->prev_value = ext->value;
break;
}
lv_cpicker_invalidate(cpicker, false);
}
static void lv_cpicker_reset_hsv_if_double_clicked(lv_obj_t * cpicker,
lv_cpicker_ext_t * ext)
{
@ -1321,18 +1311,16 @@ static void lv_cpicker_reset_hsv_if_double_clicked(lv_obj_t * cpicker,
switch(ext->color_mode)
{
case LV_CPICKER_COLOR_MODE_HUE:
ext->hue = LV_CPICKER_DEF_HUE;
ext->prev_hue = ext->hue;
ext->hsv.h = LV_CPICKER_DEF_HSV.h;
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
ext->saturation = LV_CPICKER_DEF_SATURATION;
ext->prev_saturation = ext->saturation;
ext->hsv.s = LV_CPICKER_DEF_HSV.s;
break;
case LV_CPICKER_COLOR_MODE_VALUE:
ext->value = LV_CPICKER_DEF_VALUE;
ext->prev_value = ext->value;
ext->hsv.v = LV_CPICKER_DEF_HSV.v;
break;
}
ext->prev_hsv = ext->hsv;
lv_cpicker_invalidate(cpicker, false);
}
ext->last_click = lv_tick_get();
@ -1341,21 +1329,8 @@ static void lv_cpicker_reset_hsv_if_double_clicked(lv_obj_t * cpicker,
static void lv_cpicker_set_next_color_mode(lv_obj_t * cpicker,
lv_cpicker_ext_t * ext)
{
switch(ext->color_mode)
{
case LV_CPICKER_COLOR_MODE_HUE:
ext->prev_hue = ext->hue;
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
ext->prev_saturation = ext->saturation;
break;
case LV_CPICKER_COLOR_MODE_VALUE:
ext->prev_value = ext->value;
break;
}
ext->prev_hsv = ext->hsv;
ext->color_mode = (ext->color_mode + 1) % 3;
lv_cpicker_invalidate(cpicker, true);
}
@ -1371,32 +1346,30 @@ static lv_res_t lv_cpicker_set_hsv_percent(lv_obj_t * cpicker,
{
case LV_CPICKER_COLOR_MODE_HUE:
hsv = percent * 360.0;
changed = hsv != ext->hue;
changed = hsv != ext->hsv.h;
if (changed)
{
ext->hue = hsv;
ext->prev_hue = ext->hue;
ext->hsv.h = hsv;
}
break;
case LV_CPICKER_COLOR_MODE_SATURATION:
hsv = percent * 100.0;
changed = hsv != ext->saturation;
changed = hsv != ext->hsv.s;
if (changed)
{
ext->saturation = hsv;
ext->prev_saturation = ext->saturation;
ext->hsv.s = hsv;
}
break;
case LV_CPICKER_COLOR_MODE_VALUE:
hsv = percent * 100.0;
changed = hsv != ext->value;
changed = hsv != ext->hsv.v;
if (changed)
{
ext->value = hsv;
ext->prev_value = ext->value;
ext->hsv.v = hsv;
}
break;
}
ext->prev_hsv = ext->hsv;
if (changed)
{
@ -1417,7 +1390,7 @@ static lv_res_t lv_cpicker_disc_signal(lv_obj_t * cpicker, lv_signal_t sign, voi
if(sign == LV_SIGNAL_PRESSED)
{
lv_cpicker_prev_hsv_save(ext);
ext->prev_hsv = ext->hsv;
lv_indev_t * indev = param;
@ -1444,7 +1417,8 @@ static lv_res_t lv_cpicker_disc_signal(lv_obj_t * cpicker, lv_signal_t sign, voi
}
else if(sign == LV_SIGNAL_PRESS_LOST)
{
lv_cpicker_prev_hsv_restore(cpicker, ext);
ext->prev_hsv = ext->hsv;
lv_cpicker_invalidate(cpicker, false);
}
else if(sign == LV_SIGNAL_RELEASED)
{
@ -1485,7 +1459,7 @@ static lv_res_t lv_cpicker_rect_signal(lv_obj_t * cpicker, lv_signal_t sign, voi
if(sign == LV_SIGNAL_PRESSED)
{
lv_cpicker_prev_hsv_save(ext);
ext->prev_hsv = ext->hsv;
lv_indev_t * indev = param;
@ -1510,7 +1484,8 @@ static lv_res_t lv_cpicker_rect_signal(lv_obj_t * cpicker, lv_signal_t sign, voi
}
else if(sign == LV_SIGNAL_PRESS_LOST)
{
lv_cpicker_prev_hsv_restore(cpicker, ext);
ext->prev_hsv = ext->hsv;
lv_cpicker_invalidate(cpicker, false);
}
else if(sign == LV_SIGNAL_RELEASED)
{

View File

@ -32,17 +32,13 @@ extern "C" {
**********************/
/*Data of colorpicker*/
typedef struct {
uint16_t hue;
uint8_t saturation;
uint8_t value;
lv_color_hsv_t hsv;
struct
{
lv_style_t * style;
uint8_t type;
} indicator;
uint16_t prev_hue;
uint8_t prev_saturation;
uint8_t prev_value;
lv_color_hsv_t prev_hsv;
uint16_t prev_pos;
uint8_t color_mode:2;
uint8_t color_mode_fixed:1;
@ -141,6 +137,13 @@ void lv_cpicker_set_saturation(lv_obj_t * cpicker, uint8_t saturation);
*/
void lv_cpicker_set_value(lv_obj_t * cpicker, uint8_t val);
/**
* Set the current hsv of a colorpicker.
* @param cpicker pointer to colorpicker object
* @param hsv current selected hsv
*/
void lv_cpicker_set_hsv(lv_obj_t * cpicker, lv_color_hsv_t hsv);
/**
* Set the current color of a colorpicker.
* @param cpicker pointer to colorpicker object
@ -184,14 +187,14 @@ bool lv_cpicker_get_color_mode_fixed(lv_obj_t * cpicker);
* Get style of a colorpicker.
* @param cpicker pointer to colorpicker object
* @param type which style should be get
* @return style pointer to the style
* @return pointer to the style
*/
lv_style_t * lv_cpicker_get_style(const lv_obj_t * cpicker, lv_cpicker_style_t type);
/**
* Get the current hue of a colorpicker.
* @param cpicker pointer to colorpicker object
* @return hue current selected hue
* @return current selected hue
*/
uint16_t lv_cpicker_get_hue(lv_obj_t * cpicker);
@ -209,10 +212,17 @@ uint8_t lv_cpicker_get_saturation(lv_obj_t * cpicker);
*/
uint8_t lv_cpicker_get_value(lv_obj_t * cpicker);
/**
* Get the current selected hsv of a colorpicker.
* @param cpicker pointer to colorpicker object
* @return current selected hsv
*/
lv_color_hsv_t lv_cpicker_get_hsv(lv_obj_t * cpicker);
/**
* Get the current selected color of a colorpicker.
* @param cpicker pointer to colorpicker object
* @return color current selected color
* @return current selected color
*/
lv_color_t lv_cpicker_get_color(lv_obj_t * cpicker);