From fe2e25df652475712fdb23f4163f8464a84dbcd4 Mon Sep 17 00:00:00 2001
From: Dave Chamberlain <dchamberlain@edx.org>
Date: Thu, 2 Feb 2017 15:43:26 -0500
Subject: [PATCH] ECOM-6921: Added aria-modal="true" to Support Modal    
 Changed the section to a div, and converted the aria-label     to a
 labelledby attr instead (no sense in generating the string     mulitiple
 times).

---
 lms/templates/help_modal.html | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lms/templates/help_modal.html b/lms/templates/help_modal.html
index 5198e893c1..facff24c02 100644
--- a/lms/templates/help_modal.html
+++ b/lms/templates/help_modal.html
@@ -18,7 +18,7 @@ from xmodule.tabs import CourseTabList
   <a href="#help-modal" rel="leanModal" role="button">${_("Support")}</a>
 </div>
 
-<section id="help-modal" class="modal" aria-hidden="true" role="dialog" tabindex="-1" aria-label="${_("{platform_name} Support").format(platform_name=static.get_platform_name())}">
+<div id="help-modal" class="modal" aria-hidden="true" role="dialog" aria-modal="true" tabindex="-1" aria-labelledby="support-platform-name">
   <div class="inner-wrapper">
     ## TODO: find a way to refactor this
     <button class="close-modal" tabindex="0">
@@ -31,7 +31,7 @@ from xmodule.tabs import CourseTabList
     <div id="help_wrapper">
 
         <header>
-          <h2>
+          <h2 id="support-platform-name">
         ${Text(_('{platform_name} Support')).format(
             platform_name=HTML(u'<span class="edx">{}</span>').format(static.get_platform_name())
         )}
@@ -140,7 +140,7 @@ from xmodule.tabs import CourseTabList
         </p>
     </div>
   </div>
-</section>
+</div>
 
 <script type="text/javascript">
 $(document).ready(function() {