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.Add a option for use of imagecache when displaying a taxonomy image in forum list.
Taxonomy image have a such option but it effects all taxonomy images displayed all over the site, and sometimes you don't want that.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | imagecache_option_for_displaying_taxonomy_image-1071220-7.patch | 3.81 KB | Scyther |
| #3 | taxonomy.png | 79.29 KB | __Sander__ |
| #1 | advanced_forum-1071220.patch | 4 KB | Scyther |
Comments
Comment #1
Scyther CreditAttribution: Scyther commentedThis patch addes a select list for all imagecache presets on Advanced Forum settings page and uses it when displaying the taxanomy image in forum list.
Comment #2
MichelleThanks. I can't apply it right now because I've got a huge amount of uncommitted code I need to work thru first. But a quick visual review looks reasonable enough.
Assuming it works as advertised, I'll get it in before Alpha 4. If any problems arise, though, it'll be pushed to Alpha 5 because I am really hoping to get Alpha 4 out before DCC.
Michelle
Comment #3
__Sander__ CreditAttribution: __Sander__ commentedNot exactly sure whether I am addressing the right issue.
Turning on taxonomy image and the corresponding option in AF makes the forum look weird.
For example, the table heading has one column less than the table raws
Please have a look at the screenshot
Comment #4
Shadlington CreditAttribution: Shadlington commentedSooooo... This is probably pushing to alpha 5 then?
Also, in the D7 version this would probably be quite different as terms are fieldable in core.
Comment #5
MichelleGiven how badly delayed Alpha 4 has been and the limited time I have for this module in the immediate future, this is going to have to wait, sorry. :(
Michelle
Comment #6
mcdruidScyther - sorry, this looks like a great idea, but AFAICS the patch doesn't apply to 6.x-2.x-dev any more.
As the priority is now shipping a 6.x-2.0 release, I'm having to de-prioritise 3rd-party module integration features; so as desirable as this is... I'm afraid I have to close it as "won't fix".
If you - or anyone else - are desperate to get this feature in, please reopen this issue with a patch that applies against 6.x-2.x-dev.
Comment #7
Scyther CreditAttribution: Scyther commentedI would like this to get into this module, so I am willing to help with this.
New patch attached that should work.
Comment #8
Michelle@mcdruid: Just as an FYI, Scyther is one of the Author Pane maintainers. If this is something he wants, I would really appreciate it if we could make it happen. :) I love all you peeps who showed up to save my modules from my current life-chaos. :)
Comment #9
mcdruidMichelle - your input is very welcome as always :-)
I'd definitely like to get this in, but didn't have time to resurrect the patch myself, so I'm grateful to you for looking at it again Scyther; thanks!
I'll try and get this tested and committed in the next few days, and intend to include it in the beta2.
Comment #10
mcdruidThanks again Scyther - committed to 6.x-2.x-dev