search

scottcheng / cropit

A "customizable crop and zoom" jQuery plugin.
http://scottcheng.github.io/cropit/
MIT License
875 stars 304 forks source link

Height image issue #84

Closed nhutruby closed 9 years ago

nhutruby commented 9 years ago

When I upload an image with height > width many times , cropit can not load the image version 0.2.0 runs ok Thanks

scottcheng commented 9 years ago

Can you reproduce it on the demo page? If so, could you share the image you tested with?

nhutruby commented 9 years ago

van-mai-huong-2-7394-1435630545

scottcheng commented 9 years ago

It works for me with this particular image. What browser and version are you using?

nhutruby commented 9 years ago

Hi Chromium Version 44.0.2403.89 Ubuntu 14.04 (64-bit) Firefox for Ubuntu 40.0 I don't think it is because of version browsers maybe because the preview size : width : 533px, height : 300px Thanks

scottcheng commented 9 years ago

Yes I believe that's the reason -- image width (500) is smaller than preview width (533). You can either allow small image by passing in smallImage: 'allow' to allow selecting small images, or using onImageError callback to report errors.