WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#34740 closed enhancement (wontfix)

Images in Settings > Discussion > Default Avatar have empty alt attributes

Reported by: henry.wright Owned by:
Milestone: Priority: normal
Severity: normal Version: 4.3.1
Component: Administration Keywords: has-patch
Focuses: accessibility Cc:

Description

Here's a copy of one of the <img> elements:

<img alt="" src="http://0.gravatar.com/avatar/0da2f1a9340d3aa196b870f2c107a248?s=32&amp;d=wavatar&amp;r=g&amp;forcedefault=1" srcset="http://0.gravatar.com/avatar/0da2f1a9340d3aa196b870f2c107a248?s=64&amp;d=wavatar&amp;r=g&amp;forcedefault=1 2x" class="avatar avatar-32 photo" height="32" width="32">

Attachments (1)

34740.diff (850 bytes) - added by henry.wright 4 years ago.

Download all attachments as: .zip

Change History (9)

#1 @swissspidy
4 years ago

Personally I don't see a problem here. Empty alt attributes are totally valid when the image is only for presentational reasons and doesn't contain any important additional information.

In this case, screen readers would read aloud "Mystery Person", "Blank", "Mystery Man", etc. which is exactly what we want here.

@henry.wright
4 years ago

#2 @henry.wright
4 years ago

Attached is a patch to add text to the alt attribute, should you decide it's worth doing.

#3 @henry.wright
4 years ago

  • Keywords has-patch added

#4 @swissspidy
4 years ago

  • Focuses accessibility added

#5 follow-up: @henry.wright
4 years ago

@swissspidy for the record, I'm leaning towards agreeing with you. Considering the images are already described by the surrounding text, screen readers will already have the info they need. Implementing my patch might even clutter the audio (the text could be read aloud twice).

#6 in reply to: ↑ 5 @afercia
4 years ago

  • Keywords close added

Replying to henry.wright:

the text could be read aloud twice

Yep, I agree :) @henry.wright do you agree we can close the ticket?

#7 @henry.wright
4 years ago

Yes, please go ahead and close.

#8 @SergeyBiryukov
4 years ago

  • Keywords close removed
  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.