fix(fabric.Group) make addWithUpdate compatible with nested groups

[asturur]

The pr does not work great yet, needs some understanding of what is not going right.

close #6738

[asturur]

What is the issue so far:
Given the group in the example below, that is created with this code:

var rect = new fabric.Rect({fill: '#0c0', left: 0, top: 0, width: 30, height: 40});
var rect1 = new fabric.Rect({fill: '#c00', left: 30, top: 0, width: 60, height: 50});
window.group0 = new fabric.Group([rect, rect1], {left: 20, top: 30, originX: 'left', originY: 'top'});

var rect2 = new fabric.Rect({fill: '#ccc', left:40, top: 4, width: 30, height: 40});
var rect3 = new fabric.Rect({fill: '#00c', left: 80, top: 7, width: 60, height: 50});
window.group1 = new fabric.Group([rect2, rect3], {left: 40, top: 0, originX: 'left', originY: 'top'});


window.group = new fabric.Group([group0, group1], {left: 0, top: 30, originX: 'left', originY: 'top'});


canvas.add(group);

Two triangles are added as a refence of position on the canvas

We try to add the text to the grey/blue group. with the code:

window.group1.addWithUpdate(canvas.getActiveObject())
canvas.renderAll()

we obtain this:

objects are grouped, but everything moved from its position.

[asturur]

if i overlap a copy of the group over the new one

Seems like the text has been added in correct position and what failed is the outer addWithUpdate.

I think the problem is in the fact that the group logic does not expect objects to change size when being grouped/ungrouped.

[asturur]

@jameszhong2008 seems we have a solution!

[jameszhong2008]

@asturur Great job, hope this PR can be merged soon.

[jameszhong2008]

Hope this will be approve soon by reviewer who have write access

[asturur]

the pipeline from travis is stopped because we do not have credits anymore. Without pipeline i won't merge anything.
I either need them to approve us as open source project or find another ci/cd application

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[asturur]

bad bot.

[github-actions]

Code Coverage Summary

> [email protected] coverage:report /home/runner/work/fabric.js/fabric.js
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   82.72 |     76.9 |   85.66 |   82.48 |                                               
 fabric.js |   82.72 |     76.9 |   85.66 |   82.48 | ...,29540,29665,29745-29810,29933,30032,30249 
-----------|---------|----------|---------|---------|-----------------------------------------------

[github-actions]

Code Coverage Summary

> [email protected] coverage:report /home/runner/work/fabric.js/fabric.js
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   82.72 |     76.9 |   85.66 |   82.48 |                                               
 fabric.js |   82.72 |     76.9 |   85.66 |   82.48 | ...,29540,29665,29745-29810,29933,30032,30249 
-----------|---------|----------|---------|---------|-----------------------------------------------

[github-actions]

Code Coverage Summary

> [email protected] coverage:report /home/runner/work/fabric.js/fabric.js
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   82.72 |    76.91 |   85.66 |   82.48 |                                               
 fabric.js |   82.72 |    76.91 |   85.66 |   82.48 | ...,29544,29669,29749-29814,29937,30036,30253 
-----------|---------|----------|---------|---------|-----------------------------------------------

[github-actions]

Code Coverage Summary

> [email protected] coverage:report /home/runner/work/fabric.js/fabric.js
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   82.69 |     76.9 |    85.6 |   82.45 |                                               
 fabric.js |   82.69 |     76.9 |    85.6 |   82.45 | ...,29559,29684,29764-29829,29952,30051,30268 
-----------|---------|----------|---------|---------|-----------------------------------------------

[github-actions]

Code Coverage Summary

> [email protected] coverage:report /home/runner/work/fabric.js/fabric.js
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |    82.7 |    76.91 |    85.6 |   82.46 |                                               
 fabric.js |    82.7 |    76.91 |    85.6 |   82.46 | ...,29558,29683,29763-29828,29951,30050,30267 
-----------|---------|----------|---------|---------|-----------------------------------------------