Remove offscreen Heading and cleanup course card markup

Description

Location: My Courses Dashboard https://courses.edx.org/dashboard

Severity: 3

Description: There are multiple heading elements hidden off-screen (probably for screen readers use) that are not necessary. The page has 2 "My Courses" headings implemented with H2 elements one after the other. In addition, there are a "Course Details" H3 heading before each course listed in the Dashboard. This is also unnecessary as the titles of the courses are within heading elements as well. Screen reader users who rely on these heading markups to understand the page structure may be confused when unnecessary headings are used throughout the page.

Pattern: None

Auditor Note: Developers must remove the extra heading H2 for My Courses. Only the on-screen heading should remain. The course details headings for each course should also be removed as it is not useful to screen reader users.

Internal Note: This is a general pattern I have been encouraging here at edX. Use Offscreen text sparingly, especially for things like headings. If it's helpful to add a heading for an SR user, it's probably helpful for a sighted user as well. In this case, the Offscreen H2 that precedes each course is unnecessary and should be removed. This leaves a very logical heading structure: h2 My Courses -> h3 My course name 1, h3 My course name 2, etc.

There is some additional cleanup that needs to be performed here. There is an aria-labelledby element on the section elements that references the heading that is slated to be removed by this ticket. 1) this shouldn't be a section element (div is fine), and it shouldn't have an aria-labelledby. 2) The parent article element should be a section element and it should have an aria-labelledby that references the h3 that is the course name (which will need an IDref now).

Steps to Reproduce

None

Current Behavior

None

Expected Behavior

None

Reason for Variance

None

Release Notes

None

User Impact Summary

None

Status

Assignee

Christopher Lee

Reporter

Mark Sadecki

Reach

None

Impact

None

Platform Area

None

Customer

None

Partner Manager

None

URL

None

Contributor Name

None

Groups with Read-Only Access

None

Actual Points

None

Category of Work

None

Platform Map Area (Levels 1 & 2)

None

Platform Map Area (Levels 3 & 4)

None

Story Points

2

Sprint

None

Due date

2017/12/08

Priority

CAT-2
Configure