[wp-trac] [WordPress Trac] #46124: Allow setting custom image alt text in custom header image

[WordPress Trac]

#46124: Allow setting custom image alt text in custom header image
-------------------------------------+----------------------------
 Reporter:  webmandesign             |       Owner:  audrasjb
     Type:  defect (bug)             |      Status:  accepted
 Priority:  normal                   |   Milestone:  5.9
Component:  Media                    |     Version:  5.0.3
 Severity:  normal                   |  Resolution:
 Keywords:  needs-testing has-patch  |     Focuses:  accessibility
-------------------------------------+----------------------------

Comment (by sabernhardt):

 The patch is revised to prefer user-defined alt text and fall back to the
 empty attribute.

 In #21389, the site name was chosen to avoid empty links because Twenty
 Sixteen had a linked header image. However, this function does not affect
 Twenty Sixteen; it would continue to include the site name as alt text.

 [https://wpdirectory.net/search/01FHJMHA11ET8WF8DSKXFQZA08 Four of these
 76 themes have an option to link the image], using the
 `get_header_image_tag` filter.

 Looking at dozens of themes under the following searches, I did not notice
 any others that would create an empty link with this change.
 [https://wpdirectory.net/search/01FHM82GBY6A2B7P1TS788FHM5
 the_custom_header_markup]
 [https://wpdirectory.net/search/01FHM83VW3JRXBEJZTDBYY4ACE
 get_custom_header_markup]
 [https://wpdirectory.net/search/01FHE3M7240TZA3F1V4XGF78GK
 get_header_image]
 [https://wpdirectory.net/search/01FHTR3Y3KDF165KADP0Q91K98
 get_custom_header]

 If the site name is still a safer fallback, the patch could be updated
 again.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/46124#comment:44>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform