Commit 1a19ad02 by Ahsan Ulhaq

<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
parent 2ff863cb
...@@ -6,8 +6,8 @@ ...@@ -6,8 +6,8 @@
<div class="wrapper wrapper-footer"> <div class="wrapper wrapper-footer">
<footer> <footer>
<div class="colophon"> <nav class="colophon">
<nav class="nav-colophon"> <div class="nav-colophon" aria-label="${_('Colophon')}">
<ol> <ol>
<li class="nav-colophon-01"> <li class="nav-colophon-01">
<a id="about" href="${marketing_link('ABOUT')}"> <a id="about" href="${marketing_link('ABOUT')}">
...@@ -39,7 +39,7 @@ ...@@ -39,7 +39,7 @@
</a> </a>
</li> </li>
</ol> </ol>
</nav> </div>
<div class="wrapper-logo"> <div class="wrapper-logo">
<p> <p>
...@@ -61,7 +61,7 @@ ...@@ -61,7 +61,7 @@
link_end=u"</a>" link_end=u"</a>"
)} )}
</p> </p>
<nav class="nav-legal"> <nav class="nav-legal" aria-label="${_('Legal Documents')}">
<ul> <ul>
%if marketing_link('HONOR') and marketing_link('HONOR') != '#': %if marketing_link('HONOR') and marketing_link('HONOR') != '#':
<li class="nav-legal-01"> <li class="nav-legal-01">
...@@ -89,7 +89,7 @@ ...@@ -89,7 +89,7 @@
</li> </li>
</ul> </ul>
</nav> </nav>
</div> </nav>
## please leave this link and use one of the logos provided ## please leave this link and use one of the logos provided
<div class="powered-by"> <div class="powered-by">
......
...@@ -36,8 +36,8 @@ site_status_msg = get_site_status_msg(course_id) ...@@ -36,8 +36,8 @@ site_status_msg = get_site_status_msg(course_id)
% endif % endif
</%block> </%block>
<header class="global ${"slim" if course else ""}" aria-label="${_('Global Navigation')}"> <header id="global-navigation" class="global ${"slim" if course else ""}" aria-label="${_('Global Navigation')}">
<nav> <nav aria-labelledby="global-navigation">
<h1 class="logo"> <h1 class="logo">
<a href="${marketing_link('ROOT')}"> <a href="${marketing_link('ROOT')}">
<%block name="navigation_logo"> <%block name="navigation_logo">
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment