Skip to content
This repository has been archived by the owner on Aug 1, 2020. It is now read-only.

Crop box can be moved outside image #218

Closed
max-mykhailenko opened this issue Mar 4, 2015 · 6 comments
Closed

Crop box can be moved outside image #218

max-mykhailenko opened this issue Mar 4, 2015 · 6 comments

Comments

@max-mykhailenko
Copy link

getImageData returns (it's correct)

height: 250
left: 34.64173640167364
top: 0
width: 240.71652719665272

getCropBoxData returns (it's correct)

height: 93
left: 9
top: 136
width: 93

But crop box is outside image (it's bug)
screen shot 2015-03-04 at 4 59 19 pm

@fengyuanchen
Copy link
Owner

It's the cause of EXIF Orientation, see #120.

@maiaz
Copy link

maiaz commented Mar 5, 2015

I have same issue with images without any EXIF data. basically, with any image it is possible for cropbox go out of picture boundaries... does any possibility exists to prevent this?

@MikeMatusz
Copy link

Also seeing the same thing after updating to the latest version. It's nice that you can click outside the image boundary to start your selection, which wasn't possible in the older version, but the crop box itself should still be confined to the boundaries of the image.

@rossbearman
Copy link

This is the same issue as mentioned in #207. It makes the latest version of Cropper much more difficult to use for a lot of common use cases.

@max-mykhailenko
Copy link
Author

getCropBoxData returns incorrect left parameter. left in example above must be minus ~-20

@fengyuanchen
Copy link
Owner

I had improved this in the v0.9.0.

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

No branches or pull requests

5 participants