Merge pull request #12744 from edx/clrux/ac-455

AC-455 adding main to PDF books; styling chapters

lms/static/sass/course/_textbook.scss

@@ -27,20 +27,15 @@ div.book-wrapper {
     border-right: none;
     width: 180px;
 
-    ul#booknav {
-      font-size: em(14);
-
-      .chapter-number {
-      }
+    #booknav {
+        list-style: none;
 
       .chapter {
-        float: left;
-        width: 87%;
         line-height: 1.4em;
       }
 
       .page-number {
-        float: right;
+        @include float(right);
         width: 12%;
         font-size: .8em;
         line-height: 2.1em;
@@ -59,7 +54,6 @@ div.book-wrapper {
           @include clearfix();
           padding: 0;
           color: $link-color;
-          cursor: pointer;
 
           &:hover, &:focus {
             background-color: transparent;
@@ -231,6 +225,3 @@ div.book-wrapper {
     }
   }
 }
-
-
-

lms/templates/static_htmlbook.html

@@ -118,6 +118,7 @@ from openedx.core.djangolib.js_utils import (
 <%include file="/courseware/course_navigation.html" args="active_page='htmltextbook/{0}'.format(book_index)" />
 
     <div id="outerContainer">
+        <main id="main" aria-label="${_('Content')}" tabindex="-1">
           <div id="mainContainer" class="book-wrapper">
 
             %if 'chapters' in textbook:
@@ -146,5 +147,5 @@ from openedx.core.djangolib.js_utils import (
     		<span class="idU" style="display:none">${student.id}</span>
     		<span class="idDU" style="display:none">${student.username}</span>
           </div>
+      </main>
     </div>
-

lms/templates/static_pdfbook.html

@@ -26,7 +26,8 @@ $(function(){
 });
 </script>
 
-<div class="book-wrapper">
+<main id="main" aria-label="${_('Content')}" tabindex="-1">
+    <div class="book-wrapper">
     %if 'chapters' in textbook:
     <section class="book-sidebar" aria-label="${_('Textbook Navigation')}">
         <ul id="booknav">
@@ -39,8 +40,8 @@ $(function(){
     </section>
     %endif
 
-<div class="book">
-<iframe
+    <div class="book">
+    <iframe
     title="${current_chapter['title']|h}"
     id="viewer-frame"
     src="${request.path}?viewer=true${viewer_params}"
@@ -48,5 +49,6 @@ $(function(){
     height="1108"
     frameborder="0"
     seamless></iframe>
-</div>
-</div>
+    </div>
+    </div>
+</main>

lms/templates/staticbook.html

@@ -76,6 +76,7 @@ $("#open_close_accordion a").click(function(){
 <%include file="/courseware/course_navigation.html" args="active_page='textbook/{0}'.format(book_index)" />
 
 <section class="container">
+    <main id="main" aria-label="${_('Content')}" tabindex="-1">
   <div class="book-wrapper">
 
     <section aria-label="${_('Textbook Navigation')}" class="book-sidebar">
@@ -130,4 +131,5 @@ $("#open_close_accordion a").click(function(){
       </section>
     </section>
   </div>
+  </main>
 </section>