Cropping Thumbnails, square thumbs
cbrompton - June 17, 2008 - 16:59
| Project: | Brilliant Gallery |
| Version: | 6.x-3.5 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | pluess |
| Status: | closed |
Description
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
#1
Changed title to better reflect the request
#2
I would also be very interested in this feature!
I am using 6.x-3.0 though
subscribing
#3
I'd also like to see this feature. Subscribing!
#4
I'm also very interested
#5
I would also like to see this feature!
#6
Since I also wanted to have croped thumbnails, I added this to the code. The patch is against 6.x-3.4.
#7
@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.
#8
Thanks for the patch pluess, works as expected. Hope it will appear in the next version.
#9
thanks 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:
warning: Missing argument 5 for load_dir_to_array(), called in /path/to/my/installation/sites/all/modules/brilliant_gallery/brilliant_gallery.module on line 401 and defined in /path/to/my/installation/sites/all/modules/brilliant_gallery/functions.inc on line 39.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!!
#10
Thanks 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.
#11
Looks good to go. Committed to 6.x-3.5!
#12
Automatically closed -- issue fixed for 2 weeks with no activity.