[BUG] - CropRect has mixed up width / height ⚠️ ⚠️

[vanniktech]

Using a Pixel 6 XL.

1. Click on sample of customactivity
2. Select Camera as the source
3. Take a portrait! picture
4. Have a breakpoint in onSetImageUriComplete:

   Android-Image-Cropper/cropper/src/main/java/com/canhub/cropper/CropImageActivity.kt

   Line 274 in 85c98bb

   if (error == null) {

Expected behavior: view.cropRect should have the correct dimensions (height > width)
Actual behavior: view.cropRect mixed up height with width (I get width of 3226 & height of 1814)

[vanniktech]

If you look at the image: BitmapFactory.decodeStream(view.context.contentResolver.openInputStream(uri)) - you can also see that the orientation is messed up:

[Canato]

Hey welcome 👋 to the library and thanks for the Issue.

Can you please follow the template we have for issues? 🙏
There are many points on the template that will help me to help you.

I will close 🔴 this issue so you can reopen following the template

Is important to remember that my work here is a volunteer and to save time and have better answer help me that people use the templates.

Thanks!! 🎉

[vanniktech]

Are you serious?

[Canato]

Are you serious?

Yeah, man, to be fair, if I see an issue without the template, I already send this automated message without reading it.
I spent too much time doing free volunteer work here and dealing with many people who don't read documentation or FAQ or don't know how to raise an issue.

It is for my time management and sanity that I put this type of template to help, sadly I have a limited amount of time. No other people so far have joined me to support this 200k monthly download library, so yeah, sorry that was so abrupt.

I don't use this library in any of my projects today, and I support to help the community. I ask just for people to follow the setup I put in place so I can be more effective... as you can see, we have an open PR that I didn't have the time yet to check, and no one else did and probably won't since no one never did before.

Indeed your issue is good, just missing the library version, but as a previous contributor, please use the templates and or already drop a PR with a fix.

[vanniktech]

Thanks for reopening! Library version is latest, I tried this out on the main branch at 85c98bb

[vanniktech]

This will NOT be closed by #440