search

flyfloor / react-image-cropper

react image crop
https://braavos.me/react-image-cropper/
MIT License
174 stars 56 forks source link

crop max size #25

Open tinsaye opened 6 years ago

tinsaye commented 6 years ago

Until version 1.0.5 you ware able to define the max crop size. This was a good feature we miss in our Project. With crop(maxWidth, maxHeight) you can define the max crop size. I just used the same idea from the old implementation. Signed-off-by: Tinsaye Abye abye@studserv.uni-leipzig.de

rmeissn commented 6 years ago

@jerryshew Do you plan to merge this feature in the future? It would be very handy for my project.

flyfloor commented 6 years ago

@tinsaye @rmeissn sorry I was too busy, let you waiting so long, I write a comment about this, would you update this pull request?

rmeissn commented 6 years ago

@jerryshew Thanks for responding to this PR! We forked the library and included the requested changes into this fork, that is still on version 1.1.2 (as our project still uses React v0.15, that is not compatible with your version 1.2.0). See https://github.com/slidewiki/react-image-cropper

zeeshanjan82 commented 5 years ago

I think this is a required PR we need some way to restrict the max height and max width of the cropper.