zmike pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=0d8856b24a4ec3cff2bcf7e99cd4c5669305f71b

commit 0d8856b24a4ec3cff2bcf7e99cd4c5669305f71b
Author: Mike Blumenkrantz <[email protected]>
Date:   Thu May 23 09:31:50 2019 -0400

    elm_color_class: use legacy event for genlist selection
    
    Summary:
    this fixes the selection callback and makes the widget work again
    
    @fix
    
    Reviewers: bu5hm4n
    
    Reviewed By: bu5hm4n
    
    Subscribers: bu5hm4n, cedric, #reviewers, #committers
    
    Tags: #efl_widgets
    
    Differential Revision: https://phab.enlightenment.org/D8913
---
 src/lib/elementary/elm_color_class.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/lib/elementary/elm_color_class.c 
b/src/lib/elementary/elm_color_class.c
index 9f90ae6baf..cd85f52175 100644
--- a/src/lib/elementary/elm_color_class.c
+++ b/src/lib/elementary/elm_color_class.c
@@ -152,14 +152,14 @@ _colorclass_changed(void *data, const Efl_Event *event 
EINA_UNUSED)
 }
 
 static void
-_colorclass_reset(void *data, Evas_Object *obj, void *event_info EINA_UNUSED)
+_colorclass_reset(void *data, const Efl_Event *event EINA_UNUSED)
 {
    Colorclass_UI *cc = data;
    Colorclass color;
 
    if (!cc->current)
      {
-        efl_event_callback_stop(obj);
+        efl_event_callback_stop(event->object);
         return;
      }
    //if (cc->winid && remote_iface)
@@ -194,10 +194,10 @@ _colorclass_reset(void *data, Evas_Object *obj, void 
*event_info EINA_UNUSED)
 }
 
 static void
-_colorclass_activate(void *data, const Efl_Event *event)
+_colorclass_activate(void *data, Evas_Object *obj EINA_UNUSED, void 
*event_info)
 {
    Colorclass_UI *cc = data;
-   Elm_Object_Item *it = event->info;
+   Elm_Object_Item *it = event_info;
    Edje_Color_Class *ecc;
    Colorclass *lcc;
    Eina_List *l;
@@ -742,15 +742,15 @@ elm_color_class_editor_add(Evas_Object *obj)
    elm_scroller_policy_set(gl, ELM_SCROLLER_POLICY_OFF, 
ELM_SCROLLER_POLICY_AUTO);
    elm_genlist_mode_set(gl, ELM_LIST_COMPRESS);
    elm_object_part_content_set(ly, "elm.swallow.list", gl);
-   efl_event_callback_add
-     (gl, EFL_UI_EVENT_ITEM_SELECTED, _colorclass_activate, cc);
+   evas_object_smart_callback_add(gl, "selected", _colorclass_activate, cc);
 
    cc->reset = bt = elm_button_add(ly);
    elm_object_style_set(bt, "colorclass");
    /* FIXME: translate */
    elm_object_text_set(bt, "Reset");
    elm_object_part_content_set(ly, "elm.swallow.reset", bt);
-   evas_object_smart_callback_add(bt, "clicked", _colorclass_reset, cc);
+   efl_event_callback_add
+     (bt, EFL_UI_EVENT_CLICKED, _colorclass_reset, cc);
 
    cc->cs = cs = elm_colorselector_add(ly);
    elm_colorselector_mode_set(cs, ELM_COLORSELECTOR_COMPONENTS);

-- 


Reply via email to