fix(Canvas): flipped viewport transform coords

[ShaMan123]

Motivation

#7471

gist

When flipped (negative scale) Object.isOnScreen breaks because it tried to check if a point is within vptCoords which always returned false because when flipped tl was greater than br.

Important

I've adjusted tests.
I need you to review that they are good enough, viewport transform is confusing.
The test enhancement exposed another bug with isPartiallyOnScreen.
It's test used stale canvas state, that is why it passed. Now it fails.
The reason it fails is because the rect has a common line with canvas vpt so the intersection test running in isPartiallyOnScreen returns true. But it is wrong. isPartiallyOnScreen should return false in this case, which is unhandled.

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

[asturur]

Didn't we fix this already?
Ok this is different.

[asturur]

@ShaMan123 if you have time to move this PR in a branch i can look into test failures and finish it.

[ShaMan123]

@ShaMan123 if you have time to move this PR in a branch i can look into test failures and finish it.

@asturur ?

[ShaMan123]

Should we rebase? Update from master? I'll leave it to you.
I haven't visited this PR in a long time

[ShaMan123]

Should we rebase? Update from master? I'll leave it to you.

Cause I'm afraid of unwanted overwrite

[asturur]

one test needed to be updated, since it had a flipped viewport, and so the new function was supposed to change the test result.
The other i don't know but was effectively an edge case, so i just made less edgy.