piotr/edgetx-gx12
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(table): re-implement LV_EVENT_RELEASE handler (#81)

Browse Source 
Instead of calling `lv_table`'s handler, as this causes troubles when the selected cell is changed from within `LV_EVENT_VALUE_CHANGED`.

Fixes EdgeTX/edgetx#2619
This commit is contained in:
Raphael Coeffic
2022-10-31 21:09:51 +10:00
committed by GitHub
parent a9b7f815f2
commit d174463d97
1 changed files with 17 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
radio/src/thirdparty/libopenui/src/widgets/edgetx_table.c
Vendored
+17 -13
View File
@@ -30,19 +30,23 @@ lv_obj_t * table_create(lv_obj_t * parent)
static void table_event(const lv_obj_class_t * class_p, lv_event_t * e)
{
lv_res_t res;
lv_table_t * obj = (lv_table_t *) lv_event_get_target(e);
uint16_t col = obj->col_act;
uint16_t row = obj->row_act;
/*Call the ancestor's event handler*/
res = lv_obj_event_base(MY_CLASS, e);
if(res != LV_RES_OK) return;
lv_event_code_t code = lv_event_get_code(e);
if (code == LV_EVENT_RELEASED) {
obj->col_act = col;
obj->row_act = row;
lv_obj_invalidate((lv_obj_t *)obj);
if (code != LV_EVENT_RELEASED) {
/*Call the ancestor's event handler*/
res = lv_obj_event_base(MY_CLASS, e);
if(res != LV_RES_OK) return;
} else {
lv_obj_t * obj = lv_event_get_target(e);
lv_table_t * table = (lv_table_t *)obj;
/*From lv_table.c: handler for LV_EVENT_RELEASED*/
lv_obj_invalidate(obj);
lv_indev_t* indev = lv_indev_get_act();
lv_obj_t* scroll_obj = lv_indev_get_scroll_obj(indev);
if (table->col_act != LV_TABLE_CELL_NONE &&
table->row_act != LV_TABLE_CELL_NONE && scroll_obj == NULL) {
res = lv_event_send(obj, LV_EVENT_VALUE_CHANGED, NULL);
if (res != LV_RES_OK) return;
}
}
}