Changeset 38812 for trunk/src/wp-includes/media.php

Timestamp:

10/19/2016 03:05:51 AM (9 years ago)

Author:

joemcgill

Message:

Media: Remove alt fallbacks to improve accessibility.

This removes the fallbacks in wp_get_attachment_image() and in
wp.media.string.props which attempt to generate an alt value
from the image caption or title if an alt attribute isn't explicitly
set.

This allows for image HTML to be generated that contains an empty alt
value, i.e., alt="" which is much preferable for screen readers than
reading redundant content in the case of a caption, or when reading the
image title, which is often generated from the filename and not helpful
as alt text.

Props odie2, joedolson, rianrietveld, afercia, iamjolly, joemcgill.
Fixes #34635.

File:

• trunk/src/wp-includes/media.php (modified) (1 diff)

trunk/src/wp-includes/media.php

@@ -867 +867 @@
             'src'   => $src,
             'class' => "attachment-$size_class size-$size_class",
-            'alt'   => trim(strip_tags( get_post_meta($attachment_id, '_wp_attachment_image_alt', true) )), // Use Alt field first
+            'alt'   => trim( strip_tags( get_post_meta( $attachment_id, '_wp_attachment_image_alt', true ) ) ),
         );
-        if ( empty($default_attr['alt']) )
-            $default_attr['alt'] = trim(strip_tags( $attachment->post_excerpt )); // If not, Use the Caption
-        if ( empty($default_attr['alt']) )
-            $default_attr['alt'] = trim(strip_tags( $attachment->post_title )); // Finally, use the title
 
         $attr = wp_parse_args( $attr, $default_attr );