Bug 29038

Summary:	Accessibility: OPACUserSummary heading doesn't have sufficient contrast
Product:	Koha	Reporter:	Henry Bolshaw <bolshawh>
Component:	OPAC	Assignee:	Henry Bolshaw <bolshawh>
Status:	CLOSED FIXED	QA Contact:	Testopia <testopia>
Severity:	minor
Priority:	P5 - low	CC:	bolshawh, kyle
Version:	Main	Keywords:	accessibility
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	Trivial patch
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:	21.11.00,21.05.05
Circulation function:
Bug Depends on:
Bug Blocks:	28188
Attachments:	Bug_29038: Accessibility OPACUserSummary heading doesn't have sufficient contrast Bug 29038: Accessibility OPACUserSummary heading doesn't have sufficient contrast Bug 29038: Accessibility OPACUserSummary heading doesn't have sufficient contrast

Description Henry Bolshaw 2021-09-16 03:41:24 UTC

The "heading" of the OPACUserSummary table has bad contrast (grey script on grey background). This makes the text difficult to read and fails to meet accessibility criteria.

The Summary table is only visible if you are logged in, have some checkouts and OPACUserSummary is active.

WCAG 2.1 1.4.3 Contrast (Minimum) Level AA:

"The visual presentation of text and images of text has a contrast ratio of at least 4.5:1..."

(https://www.w3.org/WAI/WCAG21/quickref/?versions=2.0#contrast-minimum)

There are several free websites for checking the contrast ratio, e.g. https://webaim.org/resources/contrastchecker/ 

You can also use the built-in accessibility functions in Firefox and Chrome, as well as specialist accessibility browser extensions like WAVE (https://wave.webaim.org/)

Comment 1 Henry Bolshaw 2021-09-20 21:41:43 UTC Comment hidden (obsolete)

Created attachment 125079 [details] [review]
Bug_29038: Accessibility OPACUserSummary heading doesn't have sufficient contrast

This patch changes the colour of the OPAC User
Summary heading text to increase the contrast
ratio, improving accessibility and making the
text easier to read.

To test:

1. Log in to the OPAC homepage as a user with
   items checked out
2. Use an accessibility tool (e.g. Chrome's
   Lighthouse tool) to test the contrast ratio.
3. Confirm the UserSummary heading has insufficient
   contrast.
4. Apply the patch and rebuild css:
   yarn build --view opac
5. Use the accessibility tool to confirm the
   contrast ratio now meets accessibility
   requirements
6. Check that the OPAC still looks ok after
   the changes

Comment 2 Owen Leonard 2021-09-21 11:07:32 UTC

Created attachment 125094 [details] [review]
Bug 29038: Accessibility OPACUserSummary heading doesn't have sufficient contrast

This patch changes the colour of the OPAC User Summary heading text to
increase the contrast ratio, improving accessibility and making the
text easier to read.

To test:

1. Log in to the OPAC homepage as a user with items checked out
2. Use an accessibility tool (e.g. Chrome's Lighthouse tool) to test the
   contrast ratio.
3. Confirm the UserSummary heading has insufficient contrast.
4. Apply the patch and rebuild css:
   yarn build --view opac
5. Use the accessibility tool to confirm the contrast ratio now meets
   accessibility requirements
6. Check that the OPAC still looks ok after the changes

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Comment 3 Katrin Fischer 2021-09-26 12:49:49 UTC

Created attachment 125326 [details] [review]
Bug 29038: Accessibility OPACUserSummary heading doesn't have sufficient contrast

This patch changes the colour of the OPAC User Summary heading text to
increase the contrast ratio, improving accessibility and making the
text easier to read.

To test:

1. Log in to the OPAC homepage as a user with items checked out
2. Use an accessibility tool (e.g. Chrome's Lighthouse tool) to test the
   contrast ratio.
3. Confirm the UserSummary heading has insufficient contrast.
4. Apply the patch and rebuild css:
   yarn build --view opac
5. Use the accessibility tool to confirm the contrast ratio now meets
   accessibility requirements
6. Check that the OPAC still looks ok after the changes

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Comment 4 Jonathan Druart 2021-09-28 13:39:26 UTC

Pushed to master for 21.11, thanks to everybody involved!

Comment 5 Kyle M Hall (khall) 2021-10-08 12:10:49 UTC

Pushed to 21.05.x for 21.05.05