Fixed explicit label issues related to CCX accessibility

Description

  1.  

    1.  

      1. Background
        (copied verbatim from *Mark Sadecki's* CCX Accessibility audit - July 17, 2015)

"... Explicit labels are elements that programmatically associate form fields with accessible text. Many of the fields missing accessible labels in CCX have visible text labels already. They just need to be wrapped in ``<label>`` elements. ``<label>`` elements can be associated with input fields in one of two ways; an implied relationship is assumed if the input is a child of the ``<label>`` element, or, explicit relationships can be defined using a ``for`` attribute on the element that references an ``id`` attribute on the input."

*Studio Updates:* None.

*LMS Updates:* CCX Dashboard now has explicit labels to meet edX Accessibility Guidelines.

  1.  

    1.  

      1. Instructor Dashboard

  1.  

    1.  

      1.  

        1. Membership Tab

  • “Email Addresses/Usernames” textarea now has an explicit label.

  1.  

    1.  

      1. CCX Coach Dashboard

  1.  

    1.  

      1.  

        1. Schedule Tab

  • “Section” select menu now has an explicit label.

  • “Subsection” select menu now has an explicit label.

  • “Unit” select menu now has an explicit label.

  • “Start Date” now has explicit label

  • “Start Time” now has explicit label

  • “Due Date” now has explicit label

  • “Due Time” now has explicit label

  1.  

    1.  

      1.  

        1. Enrollment Tab

  • “Email Addresses/Usernames” textarea now has an explicit label.

  • “Auto Enroll” checkbox now has an explicit label.

  • “Notify users by email” checkbox now has an explicit label.

  1.  

    1.  

      1. Testing
        You can use screen reading tools like chromeVox (A chrome browser plugin) or command+F5 on mac to verify this PR.
        Also read http://edx-partner-course-staff.readthedocs.org/en/latest/getting_started/accessibility.html

Issue https://github.com/mitocw/edx-platform/issues/21
EDX https://github.com/edx/edx-platform/pull/7855
EDX https://github.com/edx/edx-platform/pull/7921
EDX https://github.com/edx/edx-platform/pull/7873

@pdpinch @pwilkins
https://github.com/mitocw/edx-platform/pull/26

Activity

Show:
Chris Rodriguez
August 6, 2015, 5:07 PM

Mark and I have both gone through it, and we are consolidating notes so as not to provide duplicates. I'll try and get this up here today. Thanks for the ping.

Sarina Canelake
August 6, 2015, 5:04 PM

or could you please finish the review on this PR? Author has been waiting 7 days since replying to your last round of feedback

Sarina Canelake
July 24, 2015, 2:01 PM

I'm not sure if you're taking over accessibility reviews for Mark while he's on vacation. If you don't have bandwidth to pick this up, or don't feel comfortable doing so, no worries. It can wait for Mark to get back.

Done

Assignee

Unassigned

Reporter

Edx Admin [Administrator]

Labels

Contributor Name

Amir Qayyum Khan

Repo

edx/edx-platform

Customer

Epic Link

None

OSCM Assignee

None

Platform Map Area (Levels 1 &amp; 2)

None

Platform Map Area (Levels 3 &amp; 4)

None

Blended Hour Utilization Percentage

None

edX Theme

None

edX Squad

None

Github Lines Added

None

Github Lines Deleted

None

Priority

Unset