Images on main splash screen should have alt attributes

[Jane Sandberg]

The splash page of both the AngJS and Ang7 web client both have several images (the Evergreen logo, as well as icons for various Evergreen tools). None of these images have the required alt tag.

Since the icons don't add any particular information, I propose that they have alt="" and role="presentation".

Since the Evergreen logo does provide additional information for sighted users, I propose that it have alt="Evergreen".

[Shula Link (slink-g)]

Fix pushed here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/slink/lp_1802594_images_on_main_splash_screen_alt_attributes

[Jane Sandberg (sandbergja)]

Thanks for doing this, Sam! It looks good to me. Two notes from eyeballing your branch:

1) It doesn't look like it modifies the new (as of 3.2) Angular client's splash page. That splash page is at Open-ILS/src/eg2/src/app/staff/splash.component.html, and would need the same alt and role attributes.

2) It looks like there is an extra commit in your branch (called initial commit) that doesn't have to do with the splash screens.

[Shula Link (slink-g)]

I'll fix 1) and resubmit; 2) is just a habit when I make a new branch to make sure everything is current.

[Jason Stephenson (jstephenson)]

Sam, 2 is a bad habit to get into. It can lead to confusion and busted code.

Please, checkout a fresh branch based on master, cherry-pick only the relevant changes for this bug into that branch, and then push that branch to the working repository.

Thanks,
Jason

[Shula Link (slink-g)]

Thanks for the help and lesson in best practices! New, clean fix is pushed here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/slink/LP1802594_Splash_Page_Alt_Text

Thanks again,
Sam

[Jane Sandberg (sandbergja)]

Works well for me, Sam. Thanks for the contribution!

Signed-off branch here: user/sandbergja/LP1802594_Splash_Page_Alt_Text_signoff

[Galen Charlton (gmc)]

Pushed to master and rel_3_2, along with a follow-up to prevent a minor regression on the "Copy" => "Item" great relabeling. Thanks, Sam and Jane!