Genlist: Dont select first item if its DISPLAY_ONLY
authorVaibhav Gupta <g.vaibhav1@samsung.com>
Fri, 28 Aug 2015 11:03:35 +0000 (20:03 +0900)
committerChunEon Park <hermet@hermet.pe.kr>
Fri, 28 Aug 2015 11:03:35 +0000 (20:03 +0900)
Summary:
Updated genlist to select first item which is not in
DISPLAY_ONLY mode

@fix

Signed-off-by: Vaibhav Gupta <g.vaibhav1@samsung.com>
Test Plan:
Following is the test to verify:
1. Create a genlist
2. Make 1st item as ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY
3. Now bring the focus to genlist by key down
4. It should bring focus on 2nd item and not 1st.

Reviewers: raster, Hermet, SanghyeonLee, singh.amitesh

Subscribers: sachin.dev

Differential Revision: https://phab.enlightenment.org/D2957

legacy/elementary/src/lib/elm_genlist.c

index 49afbd8..c27f429 100644 (file)
@@ -3204,6 +3204,14 @@ _elm_genlist_elm_widget_on_focus(Eo *obj, Elm_Genlist_Data *sd)
              eo_it = elm_genlist_first_item_get(obj);
              is_sel = EINA_TRUE;
           }
+          
+        while (eo_it)
+        {
+           ELM_GENLIST_ITEM_DATA_GET(eo_it, it);
+           if ((!_is_no_select(it)) && (!elm_object_item_disabled_get(eo_it)))
+             break;
+           eo_it = EO_OBJ(ELM_GEN_ITEM_FROM_INLIST(EINA_INLIST_GET(it)->next));
+        }
 
         if (eo_it)
           {