[wp-trac] [WordPress Trac] #46124: Allow setting custom image alt text in custom header image
WordPress Trac
noreply at wordpress.org
Thu Oct 21 08:01:10 UTC 2021
#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
More information about the wp-trac
mailing list