In D11061#210074, @jsuya wrote:

update docs

@jsuya ping?

Please check my comment.

commit dea448d40c2ab2480955a0b3776512355d68a70c (HEAD -> master, origin/master, origin/HEAD)
Author: Hermet Park <hermetpark@gmail.com>
Date: Fri Jan 3 20:51:21 2020 +0900

Looks reasonable. though it might occur the compatibility break since the logic sequence can be changed by this. We hope it's very rare.
@cedric ?

Looks in this case animation progress will be broken..??

remove unecessary code.

@bu5hm4n using transit inside is out of this topic. Probably we can make an another task for this.

@jsuya @segfaultxavi @bu5hm4n Speed with direction is vector. Obviously. we don't think it speed. Maybe rate() is better than speed in this case. Plus rewind and negative playing is conceptually different. rewind is revert the playing for the amount which have been progressed from the beginning. Negative playing is regardless of the rewind. it's just playing to opposite way. Thus, rewind in animation view is necessary for user convenient who really want to rewind animation (i.e cancel effect it might started effect animation or not.)

@bu5hm4n @segfaultxavi theoretically, speed is a scalar, there is no negative value in speed. it's not velocity, If you learned the physics. efl_player_playing_reverse_set() / efl_player_playing_rewind_set() is much clear, imo.

@bu5hm4n Looks good to me, please review this.

Please check a comment.

Please check a comment.

@jsuya @bu5hm4n IMO sector_set/get() is a convenient utility function that just manages the sector list (name : range).
This doesn't need to be implemented by each derived classes.
framecontroller could provide the base function of these methods and derived class could override its functionality (if it's necessary)
We can consider this implementation way.

Hmm does this work for smart objects source ?
Very simple solution to fix this issue, user can set the source object as scroller viewport object, not the origin content.

Please check comments.

Previously, was this feature was supported correctly?

This interface will be worthy when player supports this sector playing,

Please check comments

Some comments

Please check a comment.

Please check comments.

a simple comment.

Please check comments.

it's shame that i couldn't make it to go to edd 19, at least i tried to send one rendering guy here for sharing working core tasks but it didn't work.

Please carefully test and push it if no any compatibility breaks.

I think you can refer how we apply the matrix transform for the nodes...
Alpha needs exactly the same method.

Please check a comment.

Considered the case, it's unnecessary cache it since it won't be shared with any others.
How about this ?
evas_cache_vg_entry_create(Evas *evas, const Eina_File *file, const char *key, int w, int h, Eina_Bool cacheable);

You argue the issue comes up with threading but there is no any thread synchronization methods for the thread safety. The fundamental problem must be still alive.

good!