Incorrect crop rect when move and scale squared images

[TkachenkoViacheslav]

GKImagePicker returns incorrectly cropped image for squared images (height == width). And I have found a solution, you need to replace in GKImageCropView.m this code:

" if (self.cropSize.width > self.cropSize.height) {
scale = (self.imageToCrop.size.width < self.imageToCrop.size.height ?
MAX(scaleWidth, scaleHeight) :
MIN(scaleWidth, scaleHeight));
}else{
scale = (self.imageToCrop.size.width < self.imageToCrop.size.height ?
MIN(scaleWidth, scaleHeight) :
MAX(scaleWidth, scaleHeight));
}"

with this one ("<" replaced with "<="):

" if (self.cropSize.width > self.cropSize.height) {
scale = (self.imageToCrop.size.width <= self.imageToCrop.size.height ?
MAX(scaleWidth, scaleHeight) :
MIN(scaleWidth, scaleHeight));
}else{
scale = (self.imageToCrop.size.width <= self.imageToCrop.size.height ?
MIN(scaleWidth, scaleHeight) :
MAX(scaleWidth, scaleHeight));
}"

sorry for my english :)

[jbouaziz]

helped me a lot! thanks

[SjoerdPerfors]

Hmm still having issues on this one..

[TkachenkoViacheslav]

SjoerdPerfors, please ensure that your image is square(width==height), and if that is true, can you send me your image for testing?

[SjoerdPerfors]

Yeah I realized that too! Why is that?

What I do now (I only use the cropperViewController and have my Facebook photopicker first):

#import "UIImage+Scaling.h"

if(image.size.height > image.size.width)
image = [image imageByScalingProportionallyToSize:CGSizeMake(image.size.height,image.size.height)];
else
image = [image imageByScalingProportionallyToSize:CGSizeMake(image.size.width,image.size.width)];

to make it a square and it works. But I'm sure it's not the best solution.

[TkachenkoViacheslav]

I guess we are talking on different things.
I use GKImagePicker for cropping images from camera and albums. And because of that GKImagePicker retuns incorrect crop rect when move and scale squared images, I decided to write here how to fix that problem.
So, if your image heigth!=width initially (without transformations) or if you don't use GKImagePicker at all - I don't know how to help you, sorry.

[SjoerdPerfors]

Guess you're right! Thanks for your reply.

[aerialcombat]

In GKImageCropView.m

Changed This...
if (self.cropSize.width > self.cropSize.height) {
scale = (self.imageToCrop.size.width < self.imageToCrop.size.height ?
MAX(scaleWidth, scaleHeight) :
MIN(scaleWidth, scaleHeight));
}
else{
scale = (self.imageToCrop.size.width < self.imageToCrop.size.height ?
MIN(scaleWidth, scaleHeight) :
MAX(scaleWidth, scaleHeight));
}

To...
if (self.cropSize.width > self.cropSize.height) {
scale = (self.imageToCrop.size.width < self.imageToCrop.size.height ?
MAX(scaleWidth, scaleHeight) :
MIN(scaleWidth, scaleHeight));
}
else if (self.cropSize.width == self.cropSize.height) {
scale = MAX(scaleWidth, scaleHeight);
}
else{
scale = (self.imageToCrop.size.width < self.imageToCrop.size.height ?
MIN(scaleWidth, scaleHeight) :
MAX(scaleWidth, scaleHeight));
}

[SjoerdPerfors]

@aerialcombat BIG THANKS !! This helped me a lot !

[kuga0509]

aerialcombat - your post worked for me. I am going to update my "issue" from the board to indicate this is the fix.

[joshbernfeld]

Scale is now sometimes determined for the wrong axis because image scale should be based on image width and height, NOT crop width and height.

instead of this from the original function

if (self.cropSize.width > self.cropSize.height)

change it to this and you should be good to go.

if (self.imageToCrop.size.width > self.imageToCrop.size.height)

[kuga0509]

I had noticed the random wrong axis determination, but had not dug into it. I will put this in. Thanks for looking into it!
Jonathan
Date: Wed, 7 Aug 2013 01:43:51 -0700
From: [email protected]
To: [email protected]
CC: [email protected]
Subject: Re: [GKImagePicker] Incorrect crop rect when move and scale squared images (#23)

Scale is now sometimes determined for the wrong axis because image scale should be based on image width and height, NOT crop width and height.

instead of this from the original function

if (self.cropSize.width > self.cropSize.height)

change it to this and you should be good to go.

if (self.imageToCrop.size.width > self.imageToCrop.size.height)

—
Reply to this email directly or view it on GitHub.

[jalola]

@aerialcombat thank you :)

[micnguyen]

Just a heads up - there are a number of solutions on this thread. The post that solved my problem completely was the original poster's answer (@DvaChisla). The problem that it addressed was that when you selected a completely square image (height == width) from the library, the cropping would be completely off. I have a custom rectangle size as my crop and the final image I got originally was a square image that was completely different. The first posts's solution fixed it for me and now I achieve the result I was after. Cheers.

[maksumon]

Above codes just work for either for Portrait or Landscape. But I needed to support both type of images. So, after several hours of googling I found following solution that works well.

-(CGRect)_calcVisibleRectForCropArea{    
    CGFloat scale = self.imageView.image.size.width / self.imageView.frame.size.width;
    scale *= self.scrollView.zoomScale;

    //extract visible rect from scrollview and scale it
    CGRect visibleRect = [scrollView convertRect:scrollView.bounds toView:imageView];
    visibleRect = GKScaleRect(visibleRect, scale);

    return visibleRect;
}

Courtesy: fleshgolem@89447bf

[Richallen1]

Sweet. Thanks for this guys!

[origds]

Thank you @aerialcombat your answer fix my error.