Grouping Objects together, the fill becomes black

[Terry-Fedirko]

Just uploaded 1.5.0 from 1.4.14 and found an issue when grouping objects together. If you have 2 objects. Say a circle that is red filled and a triangle that is blue filled. When I tried to grouping these 2 objects together, the fill in both objects turn black. It does not retain the original color (red and blue or any other color other than black). Example below:

Figure #1 - Both objects are created

Figure #2 - Both objects are selected

Figure #3 - Grouping them (programmed group into the context menu)

Figure #4 - After grouping occurs, both objects turn black.

Note: this problem does NOT occur in 1.4.13.

[asturur]

If you submit a bug, please try to reduce code to the minimum to show the bug.
open a fiddle or something similar to explain it.

[asturur]

I suppose you are using some saving, loading from JSON.
if you can reproduce the bug without custom code in some jsfiddle we will reopen the issue.

[spearson2010]

I'm not sure if this is the scenario the original poster had, but it seems similar. I'm loading JSON data, and when the data passes through fabric.util.enlivenObjects, the objects within the group inherit the fill set on the group.

Here is a fiddle: http://jsfiddle.net/SxBfp/64/
Change the group fill, and it will override the fill specificed on each unit. A color or blank (no fill) value will be inherited. Completely remove the fill attribute from the group, and the objects in the group maintain their original fill.

I'm not sure if this is by design or a bug.

[sapics]

@spearson2010 Thank you for giving information!
We have changed group initialization function recently, thus, the behavior would change in 1.5.1.

This is the current kangax/master builded version http://jsfiddle.net/sapics/SxBfp/65/.
Is this a what you expected one?

[spearson2010]

The fill of each item in the group is maintained, so that works for me. Question though, if I remove the fill attribute on each item, the fill becomes black. Is this a default, or should it then inherit the red fill from the group?

[rgwoodiii]

I'm having a similar issue, but right now I'm confused on what exactly fabric.util.enlivenObjects is. Does somebody mind explaining it to me?

[asturur]

is the function that switch from object form to class instances.
Maybe is a rought exaplanation.

from {type: 'rect', width : 50.... } to fabric.Rect

[Daomaster]

Hey Guys

I have the same issue when I try to group the objects in the active group, it fill everything in black.

Any Idea why?

http://jsfiddle.net/unu2xLj9/

[creaz35]

I have the exact same problem.
Does not seem to be fixed?
Thanks

[asturur]

hi @creaz35 this is an issue from 2016.
On what version of fabric are you?