salted-herring / salted-cropper Goto Github PK

If you migrate from using the default SilverStripe Image class to using SaltedCroppableImage, you end up having to create a task to migrate these images. It'd be great if there was a task included in the module to handle this.

Currently the readme says you can do this in a SS template:

$Photo
$Photo.Cropped
$Photo.Cropped.SetWidth(100)

However, if a user has not set a crop yet (i.e. have just uploaded the image & not cropped) then .Cropped returns nothing. Instead we should advise users to call .getCropped

If a user uploads an image, sets the crop & then removes the image, the next uploaded image should have the crop reset. We shouldn't automatically inherit the crop from the previous image.

Currently when using the cropper field you have to specify a ratio like so:

SaltedUploader::create('Image', 'Image')->setCropperRatio(100/50)

There are times when the ratio is irrelevant - i.e. you want the user to just use the toll to crop the image to whatever shape they like. Currently it falls back to a 1:1 ratio - I'd like to be able to have an unrestricted cropper if I don't supply a ratio.