It's my fault. My repo was not latest.
Thank you for check.

I have a question. In the case of _transit_effect_rotation_op, is anti_alias not necessary?

This can avoid unnecessary calculations, but it does not affect what is actually represented. Therefore, it should be abandon.

fix code

modify code

@Hermet I have updated the patch. Please review again.

Update commit comment

Patch update

• It doesn't traverse every frame. In render_pre, check the container's alpha and set flag to sync_render of VG object's data.

Update comment

Update patch

Ok I understand this. Thank you for review :)

Looks good to me thanks

If there is no target (info's next or prev), elm_access finds the target once again in the focus_relation (539 line).
(I think previous diff is right.)
@kimcinoo Please check this change.

I think, If evas_cache_image_data for im2 is fail, you should be returned function before freeing "im" and dropping the image.

Looks good to me

I agree

I agree this change

Looks good to me

Please check comment

@kimcinoo You are right. Thanks :) I updated patch.

Update patch

Please check comment

Please check comment

replaced by eina_streq. thank you for reivew

rebase patch
(This patch is for testing. don't merge)

Please check my comment.

In my opinion, if Efl.Ui.Image needs to do the same, I think I can handle this on line 939. Is it only necessary to support legacy?

Abandon this patch.
The way to include stroke width in container's path bounds is wrong.
We need an API of separate VG classes that can provide this feature(actual size with stroke width area), not a local function of VG Object.

Update code

Update code

Add initialization code.

It is an eo function. why is it necessary to check null for obj?

update commit message

Update patch

• add shareable parameter

Please ignore my comment. I will update this patch as soon as possible.

I think it may be difficult to find or delete the vfd with vp applied as different vfd objects are added to the same key at vg_cache->vfd_hash.

Yes Thanks :)

@bu5hm4n No not used.

When building with json vg loader enabled, a build error occurred.

/usr/bin/ld: src/tests/evas/6440e44@@evas_suite@exe/.._.._static_libs_vg_common_vg_common_json.c.o: undefined reference to symbol 'lottie_animation_property_override'
/usr/local/lib/x86_64-linux-gnu/librlottie.so.0: error adding symbols: DSO missing from command line

Update code

Update code

already fixed

update comment

Update code