useCallback in readme example

[mkosir]

Hey 👋
thanks for an awesome library 🚀
Just wondering in basic example how much sense does it make to wrap onCropComplete into useCallback. Maybe we could consider removing it, or add some explanation on what (and when) benefits it brings, wdyt?

[ValentinH]

Indeed, you are right; it's not required.

I made it this way when I was planning to rewrite this library using hooks. In that case, I would have needed a useEffect using the onCropComplete as a dependency. Teaching the users to wrap it with useCallback would have made it safer.
However, I know have some tricks to avoid to use it as a dependency so it would be safe. Anyway, I will not rewrite to hooks anytime soon, last time I tried, it was way too hard. 🙈

[ValentinH]

I've updated all sandboxes and the Readme 🚀

[mkosir]

Awesome thx 🚀