Replace main tag with div to compliance HTML5 standard

Course outline page uses two main tags that violates modern
HTML5 standard that recommends only single main tag on a webpage.
To make it compliance with modern standard the enclosing main tag
is replaced with div which will improve accessibility.

LEARNER-4233
This commit is contained in:
uzairr
2018-03-01 17:37:04 +05:00
parent 735158ac8c
commit 1fff89b839
2 changed files with 3 additions and 3 deletions

View File

@@ -80,7 +80,7 @@ from openedx.features.portfolio_project import INCLUDE_PORTFOLIO_UPSELL_MODAL
</div>
</header>
<div class="page-content">
<main class="page-content-main">
<div class="page-content-main">
% if course_home_message_fragment:
${HTML(course_home_message_fragment.body_html())}
% endif
@@ -94,7 +94,7 @@ from openedx.features.portfolio_project import INCLUDE_PORTFOLIO_UPSELL_MODAL
% if outline_fragment:
${HTML(outline_fragment.body_html())}
% endif
</main>
</div>
<aside class="page-content-secondary course-sidebar">
% if has_goal_permission:
<div class="section section-goals ${'' if current_goal else 'hidden'}">

View File

@@ -12,7 +12,7 @@ from django.utils.translation import ugettext as _
from openedx.core.djangolib.markup import HTML, Text
%>
<main role="main" class="course-outline" id="main" tabindex="-1">
<main class="course-outline" id="main" aria-label="Content" tabindex="-1">
% if blocks.get('children'):
<ol class="block-tree">
% for section in blocks.get('children'):