fix: consider page scroll when retrieving container point

[pontusdacke]

Suggested solution for #482

Takes page scroll in account in getContainerPoint so zoom with mousewheel works correct when restrictPosition is set to false.

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 655722b:

Sandbox	Source
react-easy-crop	Configuration

[ValentinH]

Thanks for this PR. I'll have a look and play a bit with this solution. I reminds me of an issue I had to address on https://github.com/ValentinH/react-easy-sort where I had to use some properties that were already taking the scroll into account.

In the end, the issue isn't related with restrictPosition=false, it's also present when it's set to true but it's less obvious.

[ValentinH]

I checked what was done on react-easy-sort and basically, we are storing the container position in the window on every gesture start and then we use this position to translate the event point (which is relative to the window) to the position inside the container.

Relevant piece of code:

• reading the position
• saving the container position

[pontusdacke]

I applied your suggestion, storing the position on each gesture. It seem to have achieved the same results, fixing our reported issue.

[ValentinH]

Thank you for the quick fix! Merging this, it will be automatically published in a few minutes 🎉

[github-actions]

🚀 PR was released in v5.0.2 🚀