Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It would be really nice to see an option to display all the thumbnails with the same aspect ratio. In otherwords, instead of just offering the maximum width of the thumbnail, add an option to support an internal crop.
Thanks
sullenmedia.com
Comment | File | Size | Author |
---|---|---|---|
#6 | brilliant_gallery.patch | 7.29 KB | pluess |
Comments
Comment #1
jan.n CreditAttribution: jan.n commentedChanged title to better reflect the request
Comment #2
castawaybcn CreditAttribution: castawaybcn commentedI would also be very interested in this feature!
I am using 6.x-3.0 though
subscribing
Comment #3
totkoo CreditAttribution: totkoo commentedI'd also like to see this feature. Subscribing!
Comment #4
nutkenz CreditAttribution: nutkenz commentedI'm also very interested
Comment #5
Calymari CreditAttribution: Calymari commentedI would also like to see this feature!
Comment #6
pluess CreditAttribution: pluess commentedSince I also wanted to have croped thumbnails, I added this to the code. The patch is against 6.x-3.4.
Comment #7
Vacilando CreditAttribution: Vacilando commented@pluess, thanks for working on that!
Unfortunately, the -dev code has changed a lot recently (lots of fixes, improvements) -- could I please ask you to roll a patch against that version.
I will then have a look / test and if all fine commit the new version.
Comment #8
charlieobrian CreditAttribution: charlieobrian commentedThanks for the patch pluess, works as expected. Hope it will appear in the next version.
Comment #9
castawaybcn CreditAttribution: castawaybcn commentedthanks for working in such a needed feature. Unfortunately it does not seem to work at all in my case. It also throws this error in the galeries admin page:
Am I missing some dependencies or configuration options (I did check the crop checkbox and cleared caches just in case)?
EDIT:
updating to 6.x-3.5 fixed it and now it is working as expected. Thanks a lot pluess!!
Comment #10
Vacilando CreditAttribution: Vacilando commentedThanks again for the initiative and code, pluess. I've implemented it in the current dev version, with some changes (the dimension calculation in load_dir_to_array() needs to be conditional), and just a few whiles ago committed as a new dev version!
Please review and let me know whether things work as expected.
Comment #11
Vacilando CreditAttribution: Vacilando commentedLooks good to go. Committed to 6.x-3.5!