elm ctxpopup: fix auto_hide mode to work correctly
authorTaehyub Kim <taehyub.kim@samsung.com>
Tue, 7 Jun 2016 08:19:15 +0000 (17:19 +0900)
committerJean-Philippe Andre <jp.andre@samsung.com>
Tue, 7 Jun 2016 08:19:15 +0000 (17:19 +0900)
@fix

Summary: fixx auto_hide mode to work correctly

Test Plan:
1. launch elementary_test - ctxpopup
2. click normal ctxpopup and auto_hide ctxpopup sample
3. check the auto_hide function whether it is work well or not

Reviewers: Jaehyun_Cho, raster, jpeg, cedric, singh.amitesh, DaveMDS, Hermet

Reviewed By: DaveMDS

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

src/bin/elementary/test_ctxpopup.c
src/lib/elementary/elc_ctxpopup.c

index 27e5376..fbc9042 100644 (file)
@@ -50,6 +50,7 @@ _ctxpopup_item_cb(void *data EINA_UNUSED, Evas_Object *obj EINA_UNUSED, void *ev
 {
    printf("ctxpopup item selected: %s\n",
    elm_object_item_text_get(event_info));
+   elm_ctxpopup_dismiss(obj);
 }
 
 static Elm_Object_Item *
index 3ef52c5..2cfc15c 100644 (file)
@@ -938,7 +938,8 @@ _bg_clicked_cb(void *data,
 {
    ELM_CTXPOPUP_DATA_GET(data, sd);
 
-   _hide_signals_emit(data, sd->dir);
+   if (sd->auto_hide)
+     _hide_signals_emit(data, sd->dir);
 }
 
 static void