Freelinks to images should show image preview instead of title (patch included!)
lendrick - January 23, 2009 - 23:51
| Project: | freelinking |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
| Issue tags: | images |
Jump to:
Description
I wanted freelinking to display the image preview if the node type being linked to was an image, so I wrote a patch for 1.7 to do that. I'd love to see it included in the official version if that's practical.
Note that the user has the option to turn it off (and it's off by default) so it should be a safe drop-in replacement for the current version.
This of course requires the Image module for it to actually display the image, but if the module is not installed, there shouldn't be any problem.
| Attachment | Size |
|---|---|
| freelinking.module.patch | 2.88 KB |
#1
Tip: always develop against the latest HEAD version. In this case I suppose it is 1.x-dev.
#2
Yes, in retrospect, that would have probably made things a bit easier. I just had the code there and started hacking away at it without really considering that. :)
#3
Thanks!! this was helpful.
#4
following
#5
More reviews and a patch reroll would convince me to commit this, despite the feature freeze on 1.x.