OpenLiberty / iguide-microprofile-config-intro

Open Liberty MicroProfile Config Guide: https://openliberty.io/guides/microprofile-config-intro.html
Other
2 stars 2 forks source link

Updates required from Accessibility Checker #273

Closed dmorgant closed 3 years ago

dmorgant commented 3 years ago

Copied from https://github.com/OpenLiberty/iguide-microprofile-config-intro/pull/272.

The following updates were made to satisfy the errors reported by the Accessibility Checker:

1) Each sample browser within the guide (on each "page" of the guide) was getting the following errors:

image

--> Fixed by adding a 'lang' attribute, a title element, and a main element which points to a unique label within each HTML.

2) The table in the playground had the following errors:

image

--> Fixed by changing scope='column' to scope='col'

3) The error message in the pod did not meet minimum contrast coloring.

--> Fixed by changing the red part of the message to a darker red meant to match the same red used in the text editor to indicate an error occurred.


The text editors on the page were reporting this error

image

This refers to the button group... Reset, Copy, Undo, Redo, and Run....in each of the editors in the guide. Each does have a unique label within the group, but not between the 10 editors in the guide. So, this is showing as a false positive.


Finally, in the single column view, the TOC button was getting the following error:

image

--> Fixed by setting a role="button" to the TOC div. However, this div is defined in a different project and therefore will be fixed under a different issue in a different repository. See https://github.com/OpenLiberty/openliberty.io/pull/2037.