From 1a19ad02f2eaed9667013f94088a6ce96c39388d Mon Sep 17 00:00:00 2001
From: Ahsan Ulhaq <ahsan.haq@arbisoft.com>
Date: Mon, 23 Feb 2015 16:30:48 +0500
Subject: [PATCH 1/5] <nav> for the footer and navigation does not include
 aria-label

<nav> element needs to include aria-label or aria-labelledby which
causes issues for the screen readers.

TNL-1510
---
 lms/templates/footer.html     | 10 +++++-----
 lms/templates/navigation.html |  4 ++--
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/lms/templates/footer.html b/lms/templates/footer.html
index d8a4c94380..fc2d22afc8 100644
--- a/lms/templates/footer.html
+++ b/lms/templates/footer.html
@@ -6,8 +6,8 @@
 
 <div class="wrapper wrapper-footer">
   <footer>
-    <div class="colophon">
-      <nav class="nav-colophon">
+    <nav class="colophon">
+      <div class="nav-colophon" aria-label="${_('Colophon')}">
       <ol>
           <li class="nav-colophon-01">
           <a id="about" href="${marketing_link('ABOUT')}">
@@ -39,7 +39,7 @@
           </a>
           </li>
       </ol>
-      </nav>
+      </div>
 
       <div class="wrapper-logo">
         <p>
@@ -61,7 +61,7 @@
 	    link_end=u"</a>"
 	)}
       </p>
-        <nav class="nav-legal">
+        <nav class="nav-legal" aria-label="${_('Legal Documents')}">
           <ul>
 	    %if marketing_link('HONOR') and marketing_link('HONOR') != '#':
             <li class="nav-legal-01">
@@ -89,7 +89,7 @@
             </li>
           </ul>
         </nav>
-    </div>
+    </nav>
 
     ## please leave this link and use one of the logos provided
     <div class="powered-by">
diff --git a/lms/templates/navigation.html b/lms/templates/navigation.html
index 945291ae2e..da8e6f517c 100644
--- a/lms/templates/navigation.html
+++ b/lms/templates/navigation.html
@@ -36,8 +36,8 @@ site_status_msg = get_site_status_msg(course_id)
 % endif
 </%block>
 
-  <header class="global ${"slim" if course else ""}" aria-label="${_('Global Navigation')}">
-    <nav>
+  <header id="global-navigation" class="global ${"slim" if course else ""}" aria-label="${_('Global Navigation')}">
+    <nav aria-labelledby="global-navigation">
     <h1 class="logo">
       <a href="${marketing_link('ROOT')}">
         <%block name="navigation_logo">

From cdb6d513d9553b371c818708310ed4751de21bb4 Mon Sep 17 00:00:00 2001
From: Ahsan Ulhaq <ahsan.haq@arbisoft.com>
Date: Mon, 23 Feb 2015 16:43:07 +0500
Subject: [PATCH 2/5] Corrected element ordering for nav

---
 lms/templates/footer.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/lms/templates/footer.html b/lms/templates/footer.html
index fc2d22afc8..74df76caeb 100644
--- a/lms/templates/footer.html
+++ b/lms/templates/footer.html
@@ -6,8 +6,8 @@
 
 <div class="wrapper wrapper-footer">
   <footer>
-    <nav class="colophon">
-      <div class="nav-colophon" aria-label="${_('Colophon')}">
+    <div class="colophon">
+      <nav class="nav-colophon" aria-label="${_('Colophon')}">
       <ol>
           <li class="nav-colophon-01">
           <a id="about" href="${marketing_link('ABOUT')}">
@@ -39,7 +39,7 @@
           </a>
           </li>
       </ol>
-      </div>
+      </nav>
 
       <div class="wrapper-logo">
         <p>
@@ -89,7 +89,7 @@
             </li>
           </ul>
         </nav>
-    </nav>
+    </div>
 
     ## please leave this link and use one of the logos provided
     <div class="powered-by">

From 73f0b86f5fd0fd5a2968703697b32eda6a7f3569 Mon Sep 17 00:00:00 2001
From: Ahsan Ulhaq <ahsan.haq@arbisoft.com>
Date: Mon, 23 Feb 2015 19:44:21 +0500
Subject: [PATCH 3/5] Removed aria label from header and add it to nav

---
 lms/templates/navigation.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lms/templates/navigation.html b/lms/templates/navigation.html
index da8e6f517c..165019fcf7 100644
--- a/lms/templates/navigation.html
+++ b/lms/templates/navigation.html
@@ -36,8 +36,8 @@ site_status_msg = get_site_status_msg(course_id)
 % endif
 </%block>
 
-  <header id="global-navigation" class="global ${"slim" if course else ""}" aria-label="${_('Global Navigation')}">
-    <nav aria-labelledby="global-navigation">
+  <header id="global-navigation" class="global ${"slim" if course else ""}" >
+    <nav aria-label="${_('Global')}">
     <h1 class="logo">
       <a href="${marketing_link('ROOT')}">
         <%block name="navigation_logo">

From dcae3b6f86942ca2995e9dcdb136ebac7a4a0adf Mon Sep 17 00:00:00 2001
From: Ahsan Ulhaq <ahsan.haq@arbisoft.com>
Date: Mon, 23 Feb 2015 19:47:56 +0500
Subject: [PATCH 4/5] changed legal document to legal

---
 lms/templates/footer.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lms/templates/footer.html b/lms/templates/footer.html
index 74df76caeb..8bf29513db 100644
--- a/lms/templates/footer.html
+++ b/lms/templates/footer.html
@@ -61,7 +61,7 @@
 	    link_end=u"</a>"
 	)}
       </p>
-        <nav class="nav-legal" aria-label="${_('Legal Documents')}">
+        <nav class="nav-legal" aria-label="${_('Legal')}">
           <ul>
 	    %if marketing_link('HONOR') and marketing_link('HONOR') != '#':
             <li class="nav-legal-01">

From 83b4f820e8ef26507bbdd66dab03ed6328f53f0e Mon Sep 17 00:00:00 2001
From: Ahsan Ulhaq <ahsan.haq@arbisoft.com>
Date: Mon, 23 Feb 2015 19:57:10 +0500
Subject: [PATCH 5/5] changed colophon to about

---
 lms/templates/footer.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lms/templates/footer.html b/lms/templates/footer.html
index 8bf29513db..0cd20862c4 100644
--- a/lms/templates/footer.html
+++ b/lms/templates/footer.html
@@ -7,7 +7,7 @@
 <div class="wrapper wrapper-footer">
   <footer>
     <div class="colophon">
-      <nav class="nav-colophon" aria-label="${_('Colophon')}">
+      <nav class="nav-colophon" aria-label="${_('About')}">
       <ol>
           <li class="nav-colophon-01">
           <a id="about" href="${marketing_link('ABOUT')}">