bu5hm4n pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=1414b8a921cb150c48adaa3e0bbe3eb1edd17d8d

commit 1414b8a921cb150c48adaa3e0bbe3eb1edd17d8d
Author: Marcel Hollerbach <[email protected]>
Date:   Fri Jul 5 18:08:02 2019 +0200

    efl_ui_spin_button: this can also be done with the focus callback
    
    this makes the code more readable, and reduces the amount of events we
    are subscribing to.
    
    Reviewed-by: Mike Blumenkrantz <[email protected]>
    Differential Revision: https://phab.enlightenment.org/D9227
---
 src/lib/elementary/efl_ui_spin_button.c | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

diff --git a/src/lib/elementary/efl_ui_spin_button.c 
b/src/lib/elementary/efl_ui_spin_button.c
index ebdf2ca82c..aff27f93d3 100644
--- a/src/lib/elementary/efl_ui_spin_button.c
+++ b/src/lib/elementary/efl_ui_spin_button.c
@@ -437,18 +437,11 @@ _entry_activated_cb(void *data, const Efl_Event *event 
EINA_UNUSED)
 
 static void
 _entry_focus_changed_cb(void *data, const Efl_Event *event)
-{
-   if (!efl_ui_focus_object_focus_get(event->object))
-     _toggle_entry(data);
-}
-
-static void
-_text_button_clicked_cb(void *data, const Efl_Event *event EINA_UNUSED)
 {
    Efl_Ui_Spin_Button_Data *sd = efl_data_scope_get(data, MY_CLASS);
 
-   if (sd->entry_visible) return;
-   _toggle_entry(data);
+   if (efl_ui_focus_object_focus_get(event->object) && sd->entry_visible) 
return;
+     _toggle_entry(data);
 }
 
 static Eina_Bool
@@ -743,8 +736,6 @@ _efl_ui_spin_button_efl_object_constructor(Eo *obj, 
Efl_Ui_Spin_Button_Data *sd)
 
    sd->text_button = efl_add(EFL_UI_BUTTON_CLASS, obj,
                              elm_widget_element_update(obj, efl_added, 
PART_NAME_TEXT_BUTTON),
-                             efl_event_callback_add(efl_added, 
EFL_UI_EVENT_CLICKED,
-                                                    _text_button_clicked_cb, 
obj),
                              efl_event_callback_add(efl_added, 
EFL_UI_FOCUS_OBJECT_EVENT_FOCUS_CHANGED,
                                                     
_text_button_focus_changed_cb, obj),
                              efl_content_set(efl_part(obj, "efl.text_button"), 
efl_added));

-- 


Reply via email to