The Cropper library referenced in the README and in the libraries info is deprecated

Rerolled #10 to apply cleanly to dev for people using this patch via Composer.

Up and subscribe to this

Patch 17 works.

+1 RTBC. I've been using patch #17 on a site successfully since March 2021.

This will have less chance to be committed until #16 💬 The Cropper library referenced in the README and in the libraries info is deprecated Needs work has been adressed or at least commented on. jQuery is partly moved out of core and should not mandatory if not required. Also please review if this issus is able to incooperate into the already committed issue with composer.libarary.json added.

I agree with #16. Now that Core is working to remove its dependencies on jQuery, that same sentiment should extend to contrib.

Is this the issue you're referring to?

✨ Create composer.libraries.json Active

The added library is still pointing to https://github.com/fengyuanchen/cropper, which isn't maintained anymore.

Thanks for coming back on this +1 - If we agree on #16 than this issue is not RTBC and needs work. It still points to https://github.com/fengyuanchen/jquery-cropper instead of https://github.com/fengyuanchen/cropperjs but the IS clearly states:

Instead let's upgrade to use just https://github.com/fengyuanchen/cropperjs in the first place. Not the jQuery wrapper, just Cropper.js.

Also we need to take the new composer file into account here now. Patch #17 needs reroll and jQuery needs to be removed. Also the README needs to be updated regarding library installation.

MR !11 in issue #3305831 → makes Cropper.js usable without the jQuery wrapper. I changed the library detection to find cropperjs. Both MR need to be applied as patches to use and detect Cropper.js.