Attached is a patch to fix a number of (relatively minor) accessibility issues with OA - a few things to note:

  1. I tweaked all color contrast problems to adhere to WCAG 2.0 AA.
  2. The maintenance and install theme didn't set the site_slogan on install, rendering the "Install Wizard" h2 element unreadable

There's also a problem with the Open Academy Courses feature in that the table view is missing labels, causing headers to be empty and cause accessibility problems with the table. However, that's on the C3 feature server, so I'm just pointing that in this issue. When I added labels, I noticed that the table headers in the default theme probably need some styling as well.

CommentFileSizeAuthor
#2 1498448-accessibility-fixes.patch3.53 KBzeropx
openacademy_a11y.patch2.21 KBAnonymous (not verified)

Comments

mrf’s picture

Assigned: Unassigned » zeropx
zeropx’s picture

Status: Active » Needs review
StatusFileSize
new3.53 KB

The issue with the titles not showing has to do with the openacademy_courses.views_default.inc file in the openacademy_courses module.

The labels were not defined which is what the views uses to generate the table header fields.

I have attached a patch that fixes this assuming that you have already have the modules in your modules directory. I'll make note against the git repo as well.

mgifford’s picture

I'm assuming this needs a re-roll to keep up with 7.x-1.0-beta3.

mrfelton’s picture

Automatically closed -- issue fixed for 2 weeks with no activity.