search

andersthorborg / ACF-Image-Crop

98 stars 35 forks source link

Add support for aspect ratio crop #59

Closed joppuyo closed 6 years ago

joppuyo commented 6 years ago

I added support for aspect ratio crop that has been useful in my use cases.

It's similar to minimum dimensions but instead of a specific pixel number like 1280x720, you can enter an aspect ratio like 16:9. This is useful in srcset / responsive image implementations where you care more about the aspect ratio than the exact pixel count of the image.

Do you think this would be a good addition the project?

If you don't think it will be appropriate in the scope of the project, I'll be happy to create my own fork instead.

Thanks.

Kwapi commented 6 years ago

This is exactly what I've been looking for! I hope it gets merged

joppuyo commented 6 years ago

Since this was not merged in 6 months, I went ahead and created my own image crop plugin for ACF on w.org which crops to a aspect ratio instead of pixel value.