raster pushed a commit to branch efl-1.20.

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

commit 3850a903e4d1426c1686938d1bea674ebe82b442
Author: Jeonghyun Yun <[email protected]>
Date:   Wed Oct 18 20:15:12 2017 +0900

    elm_scroller: fix scrolling with key move
    
    Use content_region_show instead of content_pos_set in _key_action_move
    
    Summary:
    When user keep pressing key down or else on scroller content, scroller
    animation is lagging because of elm_interface_scrollable_content_pos_set
    by step_x or step_y value.  When focus moved to next object by press key
    down or else, content_pos_set by ecore_animator continuously.  In this
    time, content_pos_set in _key_action_move by step_x or step_y value
    caused animation lagging problem.  I fixed to use content_region_show
    instead of content_pos_set in _key_action_move for remove exist
    animator.
    
    Test Plan:
    1. elementary_test -> Scroller3
    2. Press 3 times "Append 10 Items in 3s" button
    3. focus to Item1 and keep pressing key_down
    
    Reviewers: jpeg, woohyun
    
    Subscribers: cedric, jpeg
    
    Differential Revision: https://phab.enlightenment.org/D5278
---
 src/lib/elementary/elm_scroller.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/lib/elementary/elm_scroller.c 
b/src/lib/elementary/elm_scroller.c
index d736884fa2..f51731914b 100644
--- a/src/lib/elementary/elm_scroller.c
+++ b/src/lib/elementary/elm_scroller.c
@@ -255,8 +255,7 @@ _key_action_move(Evas_Object *obj, const char *params)
      }
    else return EINA_FALSE;
 
-   elm_interface_scrollable_content_pos_set(obj, x, y, EINA_TRUE);
-
+   elm_interface_scrollable_content_region_show(obj, x, y, v_w, v_h);
 
    return EINA_TRUE;
 }

-- 


Reply via email to