Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#34740 closed enhancement (wontfix)

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

Reported by: henrywright's profile 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 10 years ago.

Download all attachments as: .zip

Change History (9)

#1 @swissspidy
10 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
10 years ago

#2 @henry.wright
10 years ago

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

#3 @henry.wright
10 years ago

  • Keywords has-patch added

#4 @swissspidy
10 years ago

  • Focuses accessibility added

#5 follow-up: @henry.wright
10 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
10 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
10 years ago

Yes, please go ahead and close.

#8 @SergeyBiryukov
10 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.