I discovered this module does not use the same link for a multi-value image field. By default it would use the same delta from the link field for each image. If there are multiple images but only one link this would result in only the first image being wrapped with the link.
I modified the module to have a new option in the formatter settings to allow using the same link for all values. Patching coming soon!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gooddesignusa created an issue. See original summary.

gooddesignusa’s picture

DYdave’s picture

Title: New option to use same link for multi value image field » Same link for multiple images: new formatter option
Status: Active » Needs review

Hi @gooddesignusa,

Thank you very much for your interest for the Image Link Formatter module and for submitting this feature request along with a patch. It's certainly greatly appreciated and I sincerely apologize for the delay of my reply.

I have tested the patch from #2, it works and the feature actually seemed to work as expected.

I personally find it a very interesting feature, especially if the combined Link field is configured for a single value.
It really makes things very straight forward from a backend standpoint (submitting a node is very easy/clear).
This feature does add to site builder's flexibility with this module.

Just one comment though, when using multiple values, the links still work and the first link URL is not forced onto every single image...
So I got kind of confused there, since the option says: "Use Same Link for all items." which led me to thinking that whatever number of links and values of the links in the field, the first value would always be the one displayed for all images.

I guess we would have to change the wording to make it clearer for users what this exactly does, or enforce the first URL to apply to all images no matter how many links are added by the user (useless case in my opinion. We'd benefit more from explaining what happens, in better words).

A better wording could be: "If a single link is provided, apply it to all images."
What do you think?

Otherwise... 7 months and no other feedback whatsoever....

Has anybody else tested this? Have you found it useful? Did it work as you expected?
I mean... no interest at all has been shown for this...

Ideally, I'd like to see at least one or two constructive users' feedbacks or reviews on how this feature was useful for their projects, if the configuration was easy, etc...
Unfortunately, I haven't personally been brought to using anything close to this feature for any of my recent projects.... so I don't have any opinion on this yet...
But I would love to hear yours :)

Please let me know if you would have any questions, objections, comments, suggestions, recommendations or concerns on any aspects of this issue, I would be glad to provide more information or explain in more details.

Any questions, feedback, testing, changes, ideas or recommendations would be highly appreciated.
Thanks to all in advance.
[Pending more reviews/testing]

Status: Needs review » Needs work
DYdave’s picture

Status: Needs work » Needs review

Module doesn't have tests.
Resubmitting same patch for manual review.