atspi: toolbar - fix cannot call select callback 98/81798/2
authorShinwoo Kim <cinoo.kim@samsung.com>
Thu, 28 Jul 2016 11:00:58 +0000 (20:00 +0900)
committerShinwoo Kim <cinoo.kim@samsung.com>
Fri, 5 Aug 2016 07:44:09 +0000 (16:44 +0900)
There is a line to check select_mode in _item_select function.
The item selected(clicked) callback is called when the select_mode is ELM_OBJECT_SELECT_MODE_NONE.
The checking select_mode should be handled at one place.

Change-Id: Ic0063cf2c6dc3fe8547c5c7ee075d49404bb55f8

src/lib/elm_toolbar.c

index dba2bbb..2bea81a 100644 (file)
@@ -4119,15 +4119,12 @@ EOLIAN Eina_Bool
 _elm_toolbar_elm_interface_atspi_selection_child_select(Eo *obj EINA_UNUSED, Elm_Toolbar_Data *pd, int child_index)
 {
    Elm_Toolbar_Item_Data *item;
-   if (pd->select_mode != ELM_OBJECT_SELECT_MODE_NONE)
+   EINA_INLIST_FOREACH(pd->items, item)
      {
-        EINA_INLIST_FOREACH(pd->items, item)
+        if (child_index-- == 0)
           {
-             if (child_index-- == 0)
-               {
-                  elm_toolbar_item_selected_set(EO_OBJ(item), EINA_TRUE);
-                  return EINA_TRUE;
-               }
+             elm_toolbar_item_selected_set(EO_OBJ(item), EINA_TRUE);
+             return EINA_TRUE;
           }
      }
    return EINA_FALSE;