forked from enlightenment/efl
gengrid: Fixed broken gengrid item focus support.
This commit is a gengrid version of 7d43205a0. Now it works so much better than before. - Fixed the separate behavior between selected item and focused item. - Fixed wrong focus set when gengrid is focused first time by mouse. - Fixed wrong scroll movement when the focus highlight is disabled. - Item selection sets that item focused. So when an item is selected, do not need to set the focus again.
This commit is contained in:
parent
8129ae20ec
commit
3aef035f1d
|
@ -489,6 +489,7 @@ _item_mouse_down_cb(void *data,
|
||||||
if (ev->button != 1) return;
|
if (ev->button != 1) return;
|
||||||
|
|
||||||
it->down = 1;
|
it->down = 1;
|
||||||
|
sd->mouse_down = EINA_TRUE;
|
||||||
it->dragging = 0;
|
it->dragging = 0;
|
||||||
evas_object_geometry_get(obj, &x, &y, NULL, NULL);
|
evas_object_geometry_get(obj, &x, &y, NULL, NULL);
|
||||||
it->dx = ev->canvas.x - x;
|
it->dx = ev->canvas.x - x;
|
||||||
|
@ -565,6 +566,7 @@ _item_mouse_up_cb(void *data,
|
||||||
sd = GG_IT(it)->wsd;
|
sd = GG_IT(it)->wsd;
|
||||||
|
|
||||||
it->down = EINA_FALSE;
|
it->down = EINA_FALSE;
|
||||||
|
sd->mouse_down = EINA_FALSE;
|
||||||
if (ev->event_flags & EVAS_EVENT_FLAG_ON_HOLD)
|
if (ev->event_flags & EVAS_EVENT_FLAG_ON_HOLD)
|
||||||
sd->on_hold = EINA_TRUE;
|
sd->on_hold = EINA_TRUE;
|
||||||
else sd->on_hold = EINA_FALSE;
|
else sd->on_hold = EINA_FALSE;
|
||||||
|
@ -1900,6 +1902,7 @@ _elm_gengrid_smart_event(Eo *obj, void *_pd, va_list *list)
|
||||||
Evas_Coord page_x = 0;
|
Evas_Coord page_x = 0;
|
||||||
Evas_Coord page_y = 0;
|
Evas_Coord page_y = 0;
|
||||||
Elm_Object_Item *it = NULL;
|
Elm_Object_Item *it = NULL;
|
||||||
|
Eina_Bool sel_ret = EINA_FALSE;
|
||||||
|
|
||||||
if (elm_widget_disabled_get(obj)) return;
|
if (elm_widget_disabled_get(obj)) return;
|
||||||
if (type != EVAS_CALLBACK_KEY_DOWN) return;
|
if (type != EVAS_CALLBACK_KEY_DOWN) return;
|
||||||
|
@ -1915,204 +1918,119 @@ _elm_gengrid_smart_event(Eo *obj, void *_pd, va_list *list)
|
||||||
if ((!strcmp(ev->key, "Left")) ||
|
if ((!strcmp(ev->key, "Left")) ||
|
||||||
((!strcmp(ev->key, "KP_Left")) && (!ev->string)))
|
((!strcmp(ev->key, "KP_Left")) && (!ev->string)))
|
||||||
{
|
{
|
||||||
if ((sd->horizontal) &&
|
|
||||||
(((evas_key_modifier_is_set(ev->modifiers, "Shift")) &&
|
|
||||||
(_item_multi_select_up(sd)))
|
|
||||||
|| (_item_single_select_up(sd))))
|
|
||||||
{
|
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
|
||||||
if (ret) *ret = EINA_TRUE;
|
|
||||||
}
|
|
||||||
else if ((!sd->horizontal) &&
|
|
||||||
(((evas_key_modifier_is_set(ev->modifiers, "Shift")) &&
|
|
||||||
(_item_multi_select_left(sd)))
|
|
||||||
|| (_item_single_select_left(sd))))
|
|
||||||
{
|
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
|
||||||
if (ret) *ret = EINA_TRUE;
|
|
||||||
}
|
|
||||||
else
|
|
||||||
x -= step_x;
|
|
||||||
|
|
||||||
if (sd->horizontal)
|
if (sd->horizontal)
|
||||||
{
|
{
|
||||||
if (_item_focus_up(sd))
|
if (evas_key_modifier_is_set(ev->modifiers, "Shift"))
|
||||||
{
|
sel_ret = _item_multi_select_up(sd);
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
if (!sel_ret)
|
||||||
if (ret) *ret = EINA_TRUE;
|
sel_ret = _item_single_select_up(sd);
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
if (ret) *ret = EINA_FALSE;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
if (_item_focus_left(sd))
|
if (evas_key_modifier_is_set(ev->modifiers, "Shift"))
|
||||||
{
|
sel_ret = _item_multi_select_left(sd);
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
if (!sel_ret)
|
||||||
if (ret) *ret = EINA_TRUE;
|
sel_ret = _item_single_select_left(sd);
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
if (ret) *ret = EINA_FALSE;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (sel_ret)
|
||||||
|
{
|
||||||
|
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
||||||
|
if (ret) *ret = EINA_TRUE;
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (ret) *ret = EINA_FALSE;
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
else if ((!strcmp(ev->key, "Right")) ||
|
else if ((!strcmp(ev->key, "Right")) ||
|
||||||
((!strcmp(ev->key, "KP_Right")) && (!ev->string)))
|
((!strcmp(ev->key, "KP_Right")) && (!ev->string)))
|
||||||
{
|
{
|
||||||
if ((sd->horizontal) &&
|
|
||||||
(((evas_key_modifier_is_set(ev->modifiers, "Shift")) &&
|
|
||||||
(_item_multi_select_down(sd)))
|
|
||||||
|| (_item_single_select_down(sd))))
|
|
||||||
{
|
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
|
||||||
if (ret) *ret = EINA_TRUE;
|
|
||||||
}
|
|
||||||
else if ((!sd->horizontal) &&
|
|
||||||
(((evas_key_modifier_is_set(ev->modifiers, "Shift")) &&
|
|
||||||
(_item_multi_select_right(sd)))
|
|
||||||
|| (_item_single_select_right(sd))))
|
|
||||||
{
|
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
|
||||||
if (ret) *ret = EINA_TRUE;
|
|
||||||
}
|
|
||||||
else
|
|
||||||
x += step_x;
|
|
||||||
|
|
||||||
if (sd->horizontal)
|
if (sd->horizontal)
|
||||||
{
|
{
|
||||||
if (_item_focus_down(sd))
|
if (evas_key_modifier_is_set(ev->modifiers, "Shift"))
|
||||||
{
|
sel_ret = _item_multi_select_down(sd);
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
if (!sel_ret)
|
||||||
if (ret) *ret = EINA_TRUE;
|
sel_ret = _item_single_select_down(sd);
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
if (ret) *ret = EINA_FALSE;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
if (_item_focus_right(sd))
|
if (evas_key_modifier_is_set(ev->modifiers, "Shift"))
|
||||||
{
|
sel_ret = _item_multi_select_right(sd);
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
if (!sel_ret)
|
||||||
if (ret) *ret = EINA_TRUE;
|
sel_ret = _item_single_select_right(sd);
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
if (ret) *ret = EINA_FALSE;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (sel_ret)
|
||||||
|
{
|
||||||
|
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
||||||
|
if (ret) *ret = EINA_TRUE;
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (ret) *ret = EINA_FALSE;
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
else if ((!strcmp(ev->key, "Up")) ||
|
else if ((!strcmp(ev->key, "Up")) ||
|
||||||
((!strcmp(ev->key, "KP_Up")) && (!ev->string)))
|
((!strcmp(ev->key, "KP_Up")) && (!ev->string)))
|
||||||
{
|
{
|
||||||
if ((sd->horizontal) &&
|
|
||||||
(((evas_key_modifier_is_set(ev->modifiers, "Shift")) &&
|
|
||||||
(_item_multi_select_left(sd)))
|
|
||||||
|| (_item_single_select_left(sd))))
|
|
||||||
{
|
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
|
||||||
if (ret) *ret = EINA_TRUE;
|
|
||||||
}
|
|
||||||
else if ((!sd->horizontal) &&
|
|
||||||
(((evas_key_modifier_is_set(ev->modifiers, "Shift")) &&
|
|
||||||
(_item_multi_select_up(sd)))
|
|
||||||
|| (_item_single_select_up(sd))))
|
|
||||||
{
|
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
|
||||||
if (ret) *ret = EINA_TRUE;
|
|
||||||
}
|
|
||||||
else
|
|
||||||
y -= step_y;
|
|
||||||
|
|
||||||
if (sd->horizontal)
|
if (sd->horizontal)
|
||||||
{
|
{
|
||||||
if (_item_focus_left(sd))
|
if (evas_key_modifier_is_set(ev->modifiers, "Shift"))
|
||||||
{
|
sel_ret = _item_multi_select_left(sd);
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
if (!sel_ret)
|
||||||
if (ret) *ret = EINA_TRUE;
|
sel_ret = _item_single_select_left(sd);
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
if (ret) *ret = EINA_FALSE;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
if (_item_focus_up(sd))
|
if (evas_key_modifier_is_set(ev->modifiers, "Shift"))
|
||||||
{
|
sel_ret = _item_multi_select_up(sd);
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
if (!sel_ret)
|
||||||
if (ret) *ret = EINA_TRUE;
|
sel_ret = _item_single_select_up(sd);
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
if (ret) *ret = EINA_FALSE;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (sel_ret)
|
||||||
|
{
|
||||||
|
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
||||||
|
if (ret) *ret = EINA_TRUE;
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (ret) *ret = EINA_FALSE;
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
else if ((!strcmp(ev->key, "Down")) ||
|
else if ((!strcmp(ev->key, "Down")) ||
|
||||||
((!strcmp(ev->key, "KP_Down")) && (!ev->string)))
|
((!strcmp(ev->key, "KP_Down")) && (!ev->string)))
|
||||||
{
|
{
|
||||||
if ((sd->horizontal) &&
|
|
||||||
(((evas_key_modifier_is_set(ev->modifiers, "Shift")) &&
|
|
||||||
(_item_multi_select_right(sd)))
|
|
||||||
|| (_item_single_select_right(sd))))
|
|
||||||
{
|
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
|
||||||
if (ret) *ret = EINA_TRUE;
|
|
||||||
}
|
|
||||||
else if ((!sd->horizontal) &&
|
|
||||||
(((evas_key_modifier_is_set(ev->modifiers, "Shift")) &&
|
|
||||||
(_item_multi_select_down(sd)))
|
|
||||||
|| (_item_single_select_down(sd))))
|
|
||||||
{
|
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
|
||||||
if (ret) *ret = EINA_TRUE;
|
|
||||||
}
|
|
||||||
else
|
|
||||||
y += step_y;
|
|
||||||
|
|
||||||
if (sd->horizontal)
|
if (sd->horizontal)
|
||||||
{
|
{
|
||||||
if (_item_focus_right(sd))
|
if (evas_key_modifier_is_set(ev->modifiers, "Shift"))
|
||||||
{
|
sel_ret = _item_multi_select_right(sd);
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
if (!sel_ret)
|
||||||
if (ret) *ret = EINA_TRUE;
|
sel_ret = _item_single_select_right(sd);
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
if (ret) *ret = EINA_FALSE;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
if (_item_focus_down(sd))
|
if (evas_key_modifier_is_set(ev->modifiers, "Shift"))
|
||||||
{
|
sel_ret = _item_multi_select_down(sd);
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
if (!sel_ret)
|
||||||
if (ret) *ret = EINA_TRUE;
|
sel_ret = _item_single_select_down(sd);
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
if (ret) *ret = EINA_FALSE;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (sel_ret)
|
||||||
|
{
|
||||||
|
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
||||||
|
if (ret) *ret = EINA_TRUE;
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (ret) *ret = EINA_FALSE;
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
else if ((!strcmp(ev->key, "Home")) ||
|
else if ((!strcmp(ev->key, "Home")) ||
|
||||||
((!strcmp(ev->key, "KP_Home")) && (!ev->string)))
|
((!strcmp(ev->key, "KP_Home")) && (!ev->string)))
|
||||||
{
|
{
|
||||||
it = elm_gengrid_first_item_get(obj);
|
it = elm_gengrid_first_item_get(obj);
|
||||||
elm_gengrid_item_bring_in(it, ELM_GENGRID_ITEM_SCROLLTO_IN);
|
|
||||||
elm_gengrid_item_selected_set(it, EINA_TRUE);
|
elm_gengrid_item_selected_set(it, EINA_TRUE);
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
||||||
if (ret) *ret = EINA_TRUE;
|
if (ret) *ret = EINA_TRUE;
|
||||||
|
@ -2122,7 +2040,6 @@ _elm_gengrid_smart_event(Eo *obj, void *_pd, va_list *list)
|
||||||
((!strcmp(ev->key, "KP_End")) && (!ev->string)))
|
((!strcmp(ev->key, "KP_End")) && (!ev->string)))
|
||||||
{
|
{
|
||||||
it = elm_gengrid_last_item_get(obj);
|
it = elm_gengrid_last_item_get(obj);
|
||||||
elm_gengrid_item_bring_in(it, ELM_GENGRID_ITEM_SCROLLTO_IN);
|
|
||||||
elm_gengrid_item_selected_set(it, EINA_TRUE);
|
elm_gengrid_item_selected_set(it, EINA_TRUE);
|
||||||
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
ev->event_flags |= EVAS_EVENT_FLAG_ON_HOLD;
|
||||||
if (ret) *ret = EINA_TRUE;
|
if (ret) *ret = EINA_TRUE;
|
||||||
|
@ -2194,6 +2111,7 @@ _elm_gengrid_smart_on_focus(Eo *obj, void *_pd EINA_UNUSED, va_list *list)
|
||||||
Eina_Bool int_ret = EINA_FALSE;
|
Eina_Bool int_ret = EINA_FALSE;
|
||||||
Elm_Gengrid_Smart_Data *sd = _pd;
|
Elm_Gengrid_Smart_Data *sd = _pd;
|
||||||
Elm_Object_Item *it = NULL;
|
Elm_Object_Item *it = NULL;
|
||||||
|
Eina_Bool is_sel = EINA_FALSE;
|
||||||
|
|
||||||
eo_do_super(obj, MY_CLASS, elm_obj_widget_on_focus(&int_ret));
|
eo_do_super(obj, MY_CLASS, elm_obj_widget_on_focus(&int_ret));
|
||||||
if (!int_ret) return;
|
if (!int_ret) return;
|
||||||
|
@ -2205,15 +2123,22 @@ _elm_gengrid_smart_on_focus(Eo *obj, void *_pd EINA_UNUSED, va_list *list)
|
||||||
if (elm_widget_focus_get(obj))
|
if (elm_widget_focus_get(obj))
|
||||||
{
|
{
|
||||||
if (sd->last_focused_item)
|
if (sd->last_focused_item)
|
||||||
elm_object_item_focus_set(sd->last_focused_item, EINA_TRUE);
|
it = sd->last_focused_item;
|
||||||
else if (sd->last_selected_item)
|
else if (sd->last_selected_item)
|
||||||
elm_object_item_focus_set(sd->last_selected_item, EINA_TRUE);
|
it = sd->last_selected_item;
|
||||||
else
|
else if (!sd->mouse_down)
|
||||||
{
|
{
|
||||||
it = elm_gengrid_first_item_get(obj);
|
it = elm_gengrid_first_item_get(obj);
|
||||||
elm_object_item_focus_set(it, EINA_TRUE);
|
is_sel = EINA_TRUE;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (it)
|
||||||
|
{
|
||||||
|
if (is_sel)
|
||||||
|
elm_gengrid_item_selected_set(it, EINA_TRUE);
|
||||||
|
else
|
||||||
|
elm_object_item_focus_set(it, EINA_TRUE);
|
||||||
}
|
}
|
||||||
_elm_widget_focus_highlight_start(obj);
|
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
|
@ -2640,7 +2565,11 @@ _item_select(Elm_Gen_Item *it)
|
||||||
sd->walking++;
|
sd->walking++;
|
||||||
if (it->func.func) it->func.func((void *)it->func.data, WIDGET(it), it);
|
if (it->func.func) it->func.func((void *)it->func.data, WIDGET(it), it);
|
||||||
if (it->generation == sd->generation)
|
if (it->generation == sd->generation)
|
||||||
evas_object_smart_callback_call(WIDGET(it), SIG_SELECTED, it);
|
{
|
||||||
|
evas_object_smart_callback_call(WIDGET(it), SIG_SELECTED, it);
|
||||||
|
elm_object_item_focus_set((Elm_Object_Item *)it, EINA_TRUE);
|
||||||
|
}
|
||||||
|
|
||||||
it->walking--;
|
it->walking--;
|
||||||
sd->walking--;
|
sd->walking--;
|
||||||
if ((sd->clear_me) && (!sd->walking))
|
if ((sd->clear_me) && (!sd->walking))
|
||||||
|
|
|
@ -109,6 +109,7 @@ struct _Elm_Gengrid_Smart_Data
|
||||||
* selection */
|
* selection */
|
||||||
Eina_Bool show_region : 1;
|
Eina_Bool show_region : 1;
|
||||||
Eina_Bool bring_in : 1;
|
Eina_Bool bring_in : 1;
|
||||||
|
Eina_Bool mouse_down : 1; /**< a flag that mouse is down on the list at the moment. this flag is set to true on mouse and reset to false on mouse up */
|
||||||
};
|
};
|
||||||
|
|
||||||
struct Elm_Gen_Item_Type
|
struct Elm_Gen_Item_Type
|
||||||
|
|
Loading…
Reference in New Issue