elm_scroller: fix scrolling with key move 31/156531/1 accepted/tizen/unified/20171025.075156 submit/tizen/20171020.120936
authorJeonghyun Yun <jh0506.yun@samsung.com>
Wed, 18 Oct 2017 22:48:46 +0000 (07:48 +0900)
committerJeonghyun Yun <jh0506.yun@samsung.com>
Wed, 18 Oct 2017 22:50:32 +0000 (07:50 +0900)
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

Change-Id: Ia112e52594ad1189fe767bcc7bb0c37a82bcc3e6
Signed-off-by: Jeonghyun Yun <jh0506.yun@samsung.com>
src/lib/elm_scroller.c

index 99e90de..1fec03a 100644 (file)
@@ -278,7 +278,8 @@ _key_action_move(Evas_Object *obj, const char *params)
      }
    else return EINA_FALSE;
 
-   eo_do(obj, elm_interface_scrollable_content_pos_set(x, y, EINA_TRUE));
+   eo_do(obj, elm_interface_scrollable_content_region_show(x, y, v_w, v_h));
+
    return EINA_TRUE;
 }