hermet pushed a commit to branch master.

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

commit 3e5bdd2d15391691abe7b6df7c5f9579b5886fdb
Author: JunsuChoi <[email protected]>
Date:   Thu Jan 2 16:57:39 2020 +0900

    elm_animation_view: Fix play/play_back behavior
    
    Summary:
    In elm_animation_view(legacy), play means forward and play_back backward.
    So change the sign of the speed value for control.
    
    Test Plan: N/A
    
    Reviewers: Hermet, smohanty, kimcinoo
    
    Reviewed By: Hermet
    
    Subscribers: cedric, #reviewers, #committers
    
    Tags: #efl
    
    Differential Revision: https://phab.enlightenment.org/D10995
---
 src/lib/elementary/efl_ui_vg_animation_eo.legacy.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/lib/elementary/efl_ui_vg_animation_eo.legacy.c 
b/src/lib/elementary/efl_ui_vg_animation_eo.legacy.c
index eca7abf616..ad99d1b8a9 100644
--- a/src/lib/elementary/efl_ui_vg_animation_eo.legacy.c
+++ b/src/lib/elementary/efl_ui_vg_animation_eo.legacy.c
@@ -70,6 +70,8 @@ elm_animation_view_frame_get(const Efl_Ui_Vg_Animation *obj)
 EAPI Eina_Bool
 elm_animation_view_play(Efl_Ui_Vg_Animation *obj)
 {
+   double speed = efl_player_playback_speed_get(obj);
+   efl_player_playback_speed_set(obj, speed < 0 ? speed * -1 : speed);
    return efl_player_playing_set(obj, EINA_TRUE);
 }
 
@@ -77,7 +79,7 @@ EAPI Eina_Bool
 elm_animation_view_play_back(Efl_Ui_Vg_Animation *obj)
 {
    double speed = efl_player_playback_speed_get(obj);
-   efl_player_playback_speed_set(obj, speed < 0 ? speed * -1 : speed);
+   efl_player_playback_speed_set(obj, speed > 0 ? speed * -1 : speed);
    return efl_player_playing_set(obj, EINA_TRUE);
 }
 

-- 


Reply via email to