It would be good to be able to use text fields from a field collection to populate the image alt and title attributes, instead of using alt and title fields from the file entity. This would allow local override of these values per-slideshow and per-revision. Patch to follow.

CommentFileSizeAuthor
#2 field_slideshow-alt-title-from-field-collection-2.patch8.99 KBjohnpitcairn
#1 field_slideshow-alt-title-from-field-collection-2024729-1.patch8.29 KBjohnpitcairn

Comments

johnpitcairn’s picture

johnpitcairn commented
Title: Select image alt and title text from text fields in field collection » Select image alt and title text from text fields in field collection (patch)
Status: Active » Needs review
StatusFileSize
new field_slideshow-alt-title-from-field-collection-2024729-1.patch8.29 KB

... and patch. Applies against 7.x-1.82, because that is later than dev (??).

johnpitcairn’s picture

johnpitcairn commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Issue summary: View changes
StatusFileSize
new field_slideshow-alt-title-from-field-collection-2.patch8.99 KB

Patch updated for 7.x-2.x.

I've had this in production for 2 years, it works well, anyone care to review?

johnpitcairn’s picture

johnpitcairn commented
Assigned: johnpitcairn » Unassigned
johnpitcairn’s picture

johnpitcairn commented
Status: Needs review » Reviewed & tested by the community

In production for ages with no issues, marking RTBC.

The last submitted patch, 1: field_slideshow-alt-title-from-field-collection-2024729-1.patch, failed testing.

johnpitcairn’s picture

johnpitcairn commented

John Pitcairn queued 2: field_slideshow-alt-title-from-field-collection-2.patch for re-testing.

John Pitcairn queued 2: field_slideshow-alt-title-from-field-collection-2.patch for re-testing.

lamp5’s picture

lamp5
Primary language Polish
Location Rzeszow
commented
Status: Reviewed & tested by the community » Closed (outdated)