cedric pushed a commit to branch master.

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

commit bc2fe6bb778b559e6c88f836d8cbdfb631b193b4
Author: Yeongjong Lee <[email protected]>
Date:   Fri Oct 6 09:46:21 2017 -0700

    elementary: fix unintentional visibility of elm_hover in combobox 
constructor
    
    Summary:
    since fd9cf1d3b628f70d66ec85a4f2433c675d9ad243, All Eo canvas objects remain
    visible by default.
    but elm_hover doesn't want to be shown in combobox constructor.
    showing elm_hover in constructor occurs unexpected behavior.
    this fixes that bug.
    
    Test Plan:
    1. elementary_test - Combobox
    2. Click the combobox to Expand the list.
    
    Reviewers: jpeg, cedric
    
    Reviewed By: cedric
    
    Subscribers: cedric
    
    Differential Revision: https://phab.enlightenment.org/D5267
    
    Signed-off-by: Cedric BAIL <[email protected]>
---
 src/lib/elementary/elc_combobox.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/lib/elementary/elc_combobox.c 
b/src/lib/elementary/elc_combobox.c
index e11feb208d..9372ffe34f 100644
--- a/src/lib/elementary/elc_combobox.c
+++ b/src/lib/elementary/elc_combobox.c
@@ -358,7 +358,8 @@ _elm_combobox_efl_object_constructor(Eo *obj, 
Elm_Combobox_Data *sd)
 
    //hover
    sd->hover = efl_add(ELM_HOVER_CLASS, sd->hover_parent,
-                       elm_obj_widget_style_set(efl_added, buf));
+                       elm_obj_widget_style_set(efl_added, buf),
+                       efl_canvas_object_legacy_ctor(efl_added));
    evas_object_layer_set(sd->hover, EVAS_LAYER_MAX);
    efl_ui_mirrored_automatic_set(sd->hover, EINA_FALSE);
    elm_hover_target_set(sd->hover, obj);

-- 


Reply via email to