cedric pushed a commit to branch master.

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

commit 4ea7effe70423b5f88f98aea45a96582669aa280
Author: Jee-Yong Um <jc9...@samsung.com>
Date:   Fri Dec 16 15:02:06 2016 -0800

    elm.hoversel: fix item size calculation
    
    Summary:
    The way to find the position of hoversel expansion is based on geometry
    currently, but it causes errors when theme is not made as expected.
    This patch makes hoversel use string to find the right position and
    calculate item width correctly when items are shorter than hoversel itself.
    
    Test Plan:
    elementary_test -to hoversel
    (error case shows in tizen mobile device when screen is rotated)
    
    Reviewers: cedric, jpeg
    
    Differential Revision: https://phab.enlightenment.org/D4484
    
    Signed-off-by: Cedric BAIL <ced...@osg.samsung.com>
---
 src/lib/elementary/elc_hoversel.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/lib/elementary/elc_hoversel.c 
b/src/lib/elementary/elc_hoversel.c
index 22331b4..745e7ff 100644
--- a/src/lib/elementary/elc_hoversel.c
+++ b/src/lib/elementary/elc_hoversel.c
@@ -282,7 +282,7 @@ _resizing_eval(Evas_Object *obj, Elm_Hoversel_Data *sd)
 
    if (sd->horizontal)
      {
-        if (xx < obj_x)
+        if (!strcmp(sd->last_location, "left"))
           {
              xx = x;
              if ((xx + ww) > obj_x)
@@ -300,7 +300,7 @@ _resizing_eval(Evas_Object *obj, Elm_Hoversel_Data *sd)
      }
    else
      {
-        if (yy < obj_y)
+        if (!strcmp(sd->last_location, "top"))
           {
              yy = y;
              if ((yy + hh) > obj_y)
@@ -328,6 +328,7 @@ _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);
 }
 

-- 


Reply via email to