you could add a force same height option where the images dont get croped but only scaled to have the same height.
this would be usefull for a strip like gallery
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | image_fixed_width_height.patch | 882 bytes | darren oh |
| #3 | image_fixed_width_height.patch.txt | 1.09 KB | gordon |
Comments
Comment #1
darren ohThe image module already works like this. It only needs to be modified to not require both height and width to be set.
Comment #2
darren ohUntil someone submits a patch, you could disable the Image Exact Sizes module and choose an image width larger than the screen in your image settings. That will make all your images the same height as long as none of your images are extremely wide and short.
Comment #3
gordon commentedHe is a patch that will do both fixed width and fixed height.
To enable this, just leave the height or width parameter of the image empty and it will calculate it.
Comment #4
mariagwyn commentedI happened on this discussion, as I needed to set single dimension image uploads. I installed the patch, and it seems to be working fine, though I have only experimented a bit. I will let you know if problems crop up.
Thanks for the unwitting help!
Maria
Comment #5
darren ohIt doesn't work for me. When I set width only, I get an error message, but the thumbnail is created. When I set height only, I get an error message and no thumbnail is created. These are the error messages:
Width Only
Height Only
Comment #6
darren ohI had the image exact sizes module installed. Without it, leaving height blank works. Leaving width blank still doesn't work.
Comment #7
gordon commentedThese are not bugs with the image module.
I have this patch running perfectly on 4.7 with E-Commece
Comment #8
darren ohThe problem was in this section of the patch:
As you can see, it says to calculate the width from the width if no width is given, whereas it should say to calculate the width from the height. It works perfectly once this is corrected. I'm attaching an updated patch.
Comment #9
darren ohThe patch has been working for me with no problem.
Comment #10
darren ohComment #11
walkah commentedcommitted. thanks Darren!
Comment #12
(not verified) commented