[Bug]: Path stroke bbox

[vitonsky]

CheckList

• I agree to follow this project's Code of Conduct
• I have read and followed the Contributing Guide
• I have read and followed the Issue Tracker Guide
• I have searched and referenced existing issues and discussions
• I am filing a BUG report.
• I have managed to reproduce the bug after upgrading to the latest version
• I have created an accurate and minimal reproduction

Version

5.2.1

In What environments are you experiencing the problem?

Chrome

Node Version (if applicable)

No response

Link To Reproduction

https://codesandbox.io/s/delicate-surf-2uq4wr

Steps To Reproduce

1. Open https://codesandbox.io/s/delicate-surf-2uq4wr or https://2uq4wr.csb.app/
2. Try to change zoom and toggle objectCaching

Expected Behavior

Figure must be circle. Any zoom changes must not affect on shape of figure

Actual Behavior

You see a critically distorted figure, it's not a circle.

If you set zoom = 1 or will disable cache, you will see a correct shape.

Error Message & Stack Trace

No response

[vitonsky]

Screenshot

[ShaMan123]

I think fabric isn't fully there in term of the arc command.
But I am guessing.
Using Circle as a workaround answers your needs?

[vitonsky]

@ShaMan123 no, it works that not only for circle, but for all shapes with rounded border, for example for rhombus

[ShaMan123]

Are you talking about stroke bounding box? #8344 ? #8040 ?

[vitonsky]

@ShaMan123 yes, it looks like. Is your PR will fix this problem?

[ShaMan123]

Should
Polyline/Polygon are fixed
Path still needs looking into
Contributions are welcome

[vitonsky]

Why this problem occur only when cache enabled?

[ShaMan123]

It isn't really.
As you can see the bbox is wrong when you select the object.
Since caching depends on bbox calculation you see the severity of the bug there as opposed to the selection box which doesn't affect the object's rendering so you may consider that as less acute.