Page MenuHomePhabricator

efl_canvas_vg_container: Initialize alpha color

Authored by jsuya on Jun 11 2019, 9:19 PM.



The vg object (container, shape, ...) follows color of VG node data.
The color of node data is not initialized. it can set the required color for each object.
vg container should be layer with an alpha value of 255.

Test Plan


Diff Detail

rEFL core/efl
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
jsuya created this revision.Jun 11 2019, 9:19 PM
jsuya requested review of this revision.Jun 11 2019, 9:19 PM
jsuya updated this revision to Diff 22698.Jun 12 2019, 12:01 AM
jsuya retitled this revision from efl_cavas_vg_container: Initialize alpha color to efl_canvas_vg_container: Initialize alpha color.

fix typo

Hermet requested changes to this revision.Jun 12 2019, 7:03 PM

Please check a comment.


I think it can be initialized in efl_canvas_vg_node class.

This revision now requires changes to proceed.Jun 12 2019, 7:03 PM
jsuya added inline comments.Jun 12 2019, 9:19 PM

That way, fill color is displayed on the shape that has only line information.
Perhaps we can set color to 0 in vg_common depending on the fill information.
But I think it is better to set only the visibility of the layer in advance.

Hermet accepted this revision.Jun 12 2019, 9:55 PM

And this doesn't initialize alpha but default color.

This revision is now accepted and ready to land.Jun 12 2019, 9:55 PM
This revision was automatically updated to reflect the committed changes.