[wp-trac] [WordPress Trac] #38768: Custom logo is missing meaningful alt attribute

WordPress Trac noreply at wordpress.org
Mon May 22 20:20:39 UTC 2017


#38768: Custom logo is missing meaningful alt attribute
---------------------------------------------+----------------------------
 Reporter:  sami.keijonen                    |       Owner:  sami.keijonen
     Type:  defect (bug)                     |      Status:  assigned
 Priority:  normal                           |   Milestone:  4.8
Component:  Themes                           |     Version:
 Severity:  normal                           |  Resolution:
 Keywords:  has-patch commit has-unit-tests  |     Focuses:  accessibility
---------------------------------------------+----------------------------

Comment (by westonruter):

 Replying to [comment:27 afercia]:
 > However, this doesn't work well when updating and already existing
 custom logo in the customizer, i.e. press "chance logo", select the same
 image, update the alt attribute: the image get inserted with the previous
 alt attribute unchanged.

 This would be expected because the control only re-renders the image when
 the setting is changed. The setting just contains the attachment ID, so if
 you select the same image then there is no detected change, as the
 attachment props aren't being listened to for changes. One way to go about
 that would be to manually trigger a setting change event when the media
 library is closed after a selection has been made.

 But the problem is deeper than that actually: the media control doesn't
 ever print out values for the `alt` attributes. See
 https://github.com/WordPress/wordpress-
 develop/blob/5ea3c91d57c17fdadbd0ef910b65c4ff8891ff87/src/wp-
 includes/customize/class-wp-customize-media-control.php#L160-L187

 So a couple things would need to change here, out of scope from improving
 the Custom Logo image's `alt` on the frontend.

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


More information about the wp-trac mailing list