From 4e32bd5abbb023e6c5daef6fa42a0d571a14b6ea Mon Sep 17 00:00:00 2001
From: christopher lee <clee@edx.org>
Date: Tue, 14 Nov 2017 14:52:49 -0500
Subject: [PATCH] Fix aria label for course details on the course dashboard

---
 .../student/tests/test_verification_status.py         |  3 ++-
 .../dashboard/_dashboard_course_listing.html          | 11 +++++------
 .../_dashboard_navigation_courses.html                |  2 --
 3 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/common/djangoapps/student/tests/test_verification_status.py b/common/djangoapps/student/tests/test_verification_status.py
index 40cbcb8433..08093f2448 100644
--- a/common/djangoapps/student/tests/test_verification_status.py
+++ b/common/djangoapps/student/tests/test_verification_status.py
@@ -371,7 +371,8 @@ class TestCourseVerificationStatus(UrlResetMixin, ModuleStoreTestCase):
         # Verify that the correct banner color is rendered
         self.assertContains(
             response,
-            "<article class=\"course {}\">".format(self.MODE_CLASSES[status])
+            "<div class=\"course {}\" aria-labelledby=\"course-title-{}\">".format(
+                self.MODE_CLASSES[status], self.course.id)
         )
 
         # Verify that the correct copy is rendered on the dashboard
diff --git a/lms/templates/dashboard/_dashboard_course_listing.html b/lms/templates/dashboard/_dashboard_course_listing.html
index e8030ea9ca..2b9764ca33 100644
--- a/lms/templates/dashboard/_dashboard_course_listing.html
+++ b/lms/templates/dashboard/_dashboard_course_listing.html
@@ -59,10 +59,9 @@ from util.course import get_link_for_about_page, get_encoded_course_sharing_utm_
           lang="${course_overview.language}"
         % endif
       >
-<article class="course${mode_class}">
+<div class="course${mode_class}" aria-labelledby="course-title-${enrollment.course_id}">
   <% course_target = reverse(course_home_url_name(course_overview.id), args=[unicode(course_overview.id)]) %>
-  <section class="details" aria-labelledby="details-heading-${course_overview.number}">
-      <h2 class="hd hd-2 sr" id="details-heading-${course_overview.number}">${_('Course details')}</h2>
+  <div class="details">
     <div class="wrapper-course-image" aria-hidden="true">
       % if show_courseware_link:
         % if not is_course_blocked:
@@ -90,7 +89,7 @@ from util.course import get_link_for_about_page, get_encoded_course_sharing_utm_
       % endif
     </div>
       <div class="wrapper-course-details">
-        <h3 class="course-title">
+        <h3 class="course-title" id="course-title-${enrollment.course_id}">
           % if show_courseware_link:
             % if not is_course_blocked:
               <a data-course-key="${enrollment.course_id}" href="${course_target}">${course_overview.display_name_with_default}</a>
@@ -264,7 +263,7 @@ from util.course import get_link_for_about_page, get_encoded_course_sharing_utm_
           </div>
         </div>
       </div>
-  </section>
+  </div>
   <footer class="wrapper-messages-primary">
     <div class="messages-list">
       % if related_programs:
@@ -411,7 +410,7 @@ from util.course import get_link_for_about_page, get_encoded_course_sharing_utm_
         % endif
     </div>
   </footer>
-</article>
+</div>
 </div>
 </li>
 <script>
diff --git a/lms/templates/learner_dashboard/_dashboard_navigation_courses.html b/lms/templates/learner_dashboard/_dashboard_navigation_courses.html
index 8dab520cfd..697ae3706a 100644
--- a/lms/templates/learner_dashboard/_dashboard_navigation_courses.html
+++ b/lms/templates/learner_dashboard/_dashboard_navigation_courses.html
@@ -6,5 +6,3 @@ from django.utils.translation import ugettext as _
 <header class="wrapper-header-courses">
   <h2 class="header-courses">${_("My Courses")}</h2>
 </header>
-
-<h2 class="hd hd-2 sr" id="courses-tab">${_('My Courses')}</h2>