The <img> tag for the Web staff client's patron photo does not include an alt attribute

[Jane Sandberg]

This is true of master as of 21 January, 2018. Setting the alt text to the localized version of "Patron photograph" seems like a simple way to fix this. Setting the alt text to the localized version of "Photograph of [Patron Name]" seems even better, although not quite as easy.

[Shula Link (slink-g)]

I'm thinking that having a null alt attribute might be a better choice since the picture itself isn't providing account information. G94 of WCAG 2.0 indicates that alternative text should provide the same information as the non-text content - as the alt text wouldn't be able to provide the visual identification aspect of the patron photo, and the image doesn't act as navigation or contain any other account data, it's not contributing the same information, it's simply stating the presence of the image.

A null alt attribute would be easy to add, and would conform to H67 of WCAG 2.0 and non-disruptively remove the image from the flow of a screen reader or other AT.

[Jane Sandberg (sandbergja)]

Very good point, Sam. I'd be good with a null alt attribute.

[Shula Link (slink-g)]

I've pushed a fix to the branch at:

user/slink/lp1744585-patron-photo-img-tag-alt-attribute

Just added the alt="" into the image tag on the t_summary.tt2 file.

[Cesar V (cesardv)]

Hi Sam, thanks for your patch!

I tested this and your markup change is definitely a good WCAG improvement.

In your Git branch, however, there were a few stray deletions to unrelated files that got committed.

Thus, I went just ahead and cherry-picked+amended your changes related to this bug and pushed them to clean/new signed-off branch:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/cesardv/slink_lp1744585-patron-photo-img-tag-alt-attribute-SIGNOFF

Just FYI... the stray deletions that I stripped from the branch were to the unrelated files below:

[deleted] Open-ILS/src/edi_translator/data/test_output/ruby_json2edi.tmp (first commit)
[deleted] Open-ILS/src/edi_translator/data/test_output/perl_json2edi.tmp (first commit)
[deleted] docs/RELEASE_NOTES_NEXT/Circulation/Patron_Email_Addresses_Now_Clickable.adoc (second commit)

[Shula Link (slink-g)]

Thanks! I'm not sure how those got into the commit, so thanks for catching them and fixing it.

[Galen Charlton (gmc)]

Pushed to master, rel_3_1, and rel_3_0. Thanks, Sam and Cesar!

Note that because this is just a bugfix and doesn't require special instructions, I did not push the release notes entry.