Group selector uses canvas coordinates which can correspond to different fabric coordinate when panning/zooming while selecting

[SLKnutson]

Version

4.5.0, don't think this has changed recently

Test Case

http://fabricjs.com/fabric-intro-part-5

Information about environment

Any browser

Steps to reproduce

1. Go to the first fabric canvas on http://fabricjs.com/fabric-intro-part-5
2. Use a mouse with a scroll wheel
3. Hold down the left mouse button
4. Move the mouse slightly to create a blue selection area
5. While keeping the mouse button held down, zoom in or out using the mouse wheel

Expected Behavior

The grouping should stay anchored to the original fabric coordinate that it was created in

Actual Behavior

The grouping is anchored to the canvas offset that it was originally created in

Reasoning

Not super obvious with the zooming in/out, but this is really evident when implementing autopanning when the cursor moves to the edge of the canvas. Theoretically someone could want it to behave the way that it currently does, but I can't imagine why.

Thanks!

[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.