cedric pushed a commit to branch master.

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

commit 3f6e8871e240d8895850d7bd18db7091a5f4662f
Author: Jee-Yong Um <jc9...@samsung.com>
Date:   Thu Jan 12 16:32:02 2017 -0800

    elm.hoversel: revert unnecessary item expansion & get proper size
    
    Summary:
    The changes of previous patch (4ea7effe70423b5f88f98aea45a96582669aa280)
    are reverted, and item calculation is fixed correctly.
    The main reason why hoversel item has wrong size in screen rotation is
    that hover doesn't update geometry when the size of target object is 
changed.
    
    Test Plan: elementary_test -to hoversel
    
    Reviewers: cedric, jpeg
    
    Differential Revision: https://phab.enlightenment.org/D4556
    
    Signed-off-by: Cedric BAIL <ced...@osg.samsung.com>
---
 src/lib/elementary/elc_hoversel.c | 1 -
 src/lib/elementary/elm_hover.c    | 4 ++++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/lib/elementary/elc_hoversel.c 
b/src/lib/elementary/elc_hoversel.c
index e95ddc6..0ecf69c 100644
--- a/src/lib/elementary/elc_hoversel.c
+++ b/src/lib/elementary/elc_hoversel.c
@@ -331,7 +331,6 @@ _resizing_eval(Evas_Object *obj, Elm_Hoversel_Data *sd)
                }
           }
      }
-   if (ww < obj_w) ww = obj_w;
    evas_object_size_hint_min_set(sd->spacer, ww, hh);
 }
 
diff --git a/src/lib/elementary/elm_hover.c b/src/lib/elementary/elm_hover.c
index 3f306ff..f39711f 100644
--- a/src/lib/elementary/elm_hover.c
+++ b/src/lib/elementary/elm_hover.c
@@ -699,6 +699,8 @@ _elm_hover_target_set(Eo *obj, Elm_Hover_Data *sd, 
Evas_Object *target)
           (sd->target, EVAS_CALLBACK_DEL, _target_del_cb, obj);
         evas_object_event_callback_del_full
           (sd->target, EVAS_CALLBACK_MOVE, _target_move_cb, obj);
+        evas_object_event_callback_del_full
+          (sd->target, EVAS_CALLBACK_RESIZE, _target_move_cb, obj);
         elm_widget_hover_object_set(sd->target, NULL);
      }
 
@@ -709,6 +711,8 @@ _elm_hover_target_set(Eo *obj, Elm_Hover_Data *sd, 
Evas_Object *target)
           (sd->target, EVAS_CALLBACK_DEL, _target_del_cb, obj);
         evas_object_event_callback_add
           (sd->target, EVAS_CALLBACK_MOVE, _target_move_cb, obj);
+        evas_object_event_callback_add
+          (sd->target, EVAS_CALLBACK_RESIZE, _target_move_cb, obj);
         elm_widget_hover_object_set(target, obj);
         elm_layout_sizing_eval(obj);
      }

-- 


Reply via email to