In the media module, we hacked the WYSIWYG to override the alt and title attributes of images. I have created a patch to make that more generic, allowing us to override whatever fields are defined for a file type, at #835516: Add ability to edit file meta data and fields when embedded in WYSIWYG.

I like the wording that was used in that form's description at #1881152: Browser display cleanup, so I have created a patch to add the description to the alt and title attributes for the image file type. I will attach that patch momentarily.

CommentFileSizeAuthor
#8 2012868-alt-title-description-8.patch2.21 KBKarlKedrovsky
#2 2012868-alt-title-description-2.patch1.08 KBaaron
#1 2012868-alt-title-description-1.patch1.08 KBaaron
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aaron’s picture

aaron CreditAttribution: aaron commented
Status: Active » Needs review
FileSize
2012868-alt-title-description-1.patch1.08 KB
aaron’s picture

aaron CreditAttribution: aaron commented
FileSize
2012868-alt-title-description-2.patch1.08 KB

Oops! Here is a good patch.

aaron’s picture

aaron CreditAttribution: aaron commented

Bump

gmclelland’s picture

gmclelland CreditAttribution: gmclelland commented

I haven't tested this yet, but the patch looks very small, simple, and helpful.

KarlKedrovsky’s picture

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

I've tested this on a fresh install of the media_dev distribution and it worked just fine.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

Should we also update the existing field instances with these descriptions if they're still empty?

HaloFX’s picture

HaloFX CreditAttribution: HaloFX commented

#2 also worked fine for me on a fresh install.

KarlKedrovsky’s picture

KarlKedrovsky CreditAttribution: KarlKedrovsky commented
Status: Reviewed & tested by the community » Needs review
FileSize
2012868-alt-title-description-8.patch2.21 KB

Attached is an updated patch that updates existing instances of the alt and title text if they haven't been changed from the previous defaults.

HaloFX’s picture

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

#8 tested on Media_dev profile and works for me!

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Reviewed & tested by the community » Fixed

Committed #8 to 7.x-2.x. Thanks all! http://drupalcode.org/project/file_entity.git/commit/e8d0faa

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.