Cropping when rotating

Hi,

I would like to know, if it’s a bug or something I did wrong : I added today the enableOrientation: true option. Working great, the image is rotating. Yay. But I have some troubles during cropping.

In some words : I’m having an image larger than higher. When I flip it to the right, the image become higher than larger. If I try to crop the bottom of the image, the result can be transparent. And because images are always better than words :

Here the image I want to upload :

Then I flip it

I select the bottom of the image

And here the result

Am I the only one who experienced that ? Let me know if it’s not clear.

Last version of Croppie : 2.4.1 Google chrome Original dimension of the image : 1000 * 667 Viewport : 200 * 200 Boundary : 260 * 260

[JonathonRichardson]

I can still reproduce this with both master and the 2.5.0. It only happens with enforceBoundary disabled and an image with an aspect ratio of something other than 1:1. It looks to me like the problem is that the original image’s width and height are used during the final drawing calculation. If the image was rotated an odd multiple of 90 degrees, that means it is using the width as the height and vice versa. I’m able to fix this so far (I haven’t tested it extensively yet) by adding the following to the end of the _rotate() function to swap the original values:

if ((deg % 180) !== 0) {
    var lastOriginalHeight = this._originalImageHeight;
    this._originalImageHeight = this._originalImageWidth;
    this._originalImageWidth = lastOriginalHeight;
}

@thedustinsmith, does that seem like the right way to fix this?

[thedustinsmith]

Thanks for all the feedback in this one guys. I believe this issue is fixed in 2.5.2. I can’t recreate the issue using the rotate example in the demo page. 2.5.2 should be released soon.