Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Library no cropping what it should #111

Closed
khrizt opened this issue May 5, 2016 · 10 comments
Closed

Library no cropping what it should #111

khrizt opened this issue May 5, 2016 · 10 comments
Labels

Comments

@khrizt
Copy link

khrizt commented May 5, 2016

Hi,

Implementing UCrop in my project I found a problem when cropping some images, I'm attaching the screenshots, hope they help:

Cropping the image

screenshot_2016-05-05-17-38-34

The result

screenshot_2016-05-05-17-38-44

Looks like rotates the image and does not get the correct coordinates. Maybe something with the meta-data of the images, sometimes I had problems with rotated images from Samsung or LG cameras.

@shliama
Copy link
Contributor

shliama commented May 5, 2016

How can I reproduce it? Do you touch/change image position before cropping?
Please attach the original image.

@khrizt
Copy link
Author

khrizt commented May 5, 2016

Sorry here is the original image and yep I change the image position before cropping

20160505_173429

Attaching the image appeared in landscape format so it looks like it's something about meta-data rotation with attribute ExifInterface.TAG_ORIENTATION.

@shliama
Copy link
Contributor

shliama commented May 5, 2016

Yeah, I could reproduce it on my device too. Seems to be issue with Exif properties.
I'll try to fix in the next update :)

@khrizt
Copy link
Author

khrizt commented May 6, 2016

Thanks man! I'll put it on my app on the next update.

@shliama
Copy link
Contributor

shliama commented Jun 27, 2016

@khrizt please try to use library version 2.1.0 - now EXIF data should be processed properly.

@shliama shliama added the bug label Jun 27, 2016
@shliama shliama self-assigned this Jun 27, 2016
@khrizt
Copy link
Author

khrizt commented Jun 27, 2016

Thanks man, I'll give it a try and tell you something

@khrizt
Copy link
Author

khrizt commented Jun 28, 2016

Looks like everything is allright, I plan on adding your library to the release this week

@khrizt khrizt closed this as completed Jun 28, 2016
@zzxxasp
Copy link

zzxxasp commented Jul 6, 2016

the bug is still exists in 2.1.0 from SonyZ5

@shliama
Copy link
Contributor

shliama commented Jul 6, 2016

@zzxxasp please, attach an image so I can reproduce the bug.

@shliama shliama reopened this Jul 6, 2016
@TeeRawk
Copy link
Contributor

TeeRawk commented Mar 6, 2017

outdated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants