It's possible to take the cropping rectangle beyond the image, if the image is smaller than the cropping canvas.
This causes problems:
- you can't enforce the aspect ratio of the crop, as the user can place the cropping rectangle overlapping the edge off the image
- it's hard to aim for an exact edge of the image if it's not on the canvas edge

It seems to me the best way to fix this might be to size the canvas according to the image to be cropped, shrinking it down for the smaller dimension.
Some exchange of information from theme_imagefield_crop_crop_image (which knows the size of the image to display) and theme_imagefield_crop_edit_crop_image_row (which shows the canvas) would be required.

Comments

stevenpatz’s picture

stevenpatz
he/him
commented
Title: cconstraing cropping rectangle to image » Constraining cropping rectangle to image
yhager’s picture

yhager commented
Status: Active » Closed (won't fix)

Thank you for your suggestion. This was actually developed as a feature to allow the user to "modify" the aspect ratio of the original image, although the resulting aspect ratio is fixed.

As the module was being used by the community, this came out to be a rarely used feature, and it was removed from the 6.x version. I'm not actively maintaining the 5.x version, but I will accept a patch if you send it. I hope this is not critical for you, and recommend you to move to Drupal 6.x anyway (imagefield_crop-6.x is much much better, not to mention Drupal itself..)