fix(geometry): nonTransformedDimensions + strokeUniform

[goodbot1356]

#7605

[ShaMan123]

Could you run npm run lint -- --fix and commit the changes (if it makes a giant commit first stash all changes and then restore stash - whitespace OS diffs)
We want to make contributing easier but for now it is what it is.

Adding a test is needed as well (something that breaks without your fix)

[asturur]

This has been proposed in the past and hasn't been done because it breaks other things.
when we talk of non transformed dimensions, the scale is not taken in consideration. is the size of the object at scale 1. so dividing by scale would bring to strange results. Getting nonTransformedDimensions of a rect when it has scale 5 or 1 should give you the same result.

All tests break, and that could be that because they are made to reflect the old behaviour, but i m not confident to accept this change because it fixes a demo.

Does it fix something else?

Let's check it more broadly. SVG import tests could be affected by this for example.

[ShaMan123]

I think there's a need to clearly define strokeUniform.
I understand it remains the same size no matter the scale.
If so is it part of non transformed dimensions?

[asturur]

strokeUniform would mean that the stroke is not affected by the current transformation.
so if you deform a path stretching it on a single axis, the stroke will stay of the same size instead of being larger in some directions.
Is not the feature wrong but our stack of transform to find the size.

[asturur]

This behaviour needs to be fixed, just to be clear, there are multiple open issues for it.

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

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