From 310073ddefc78b116f485729f90d0014c22b9179 Mon Sep 17 00:00:00 2001
From: Brian Wilson <brian@edx.org>
Date: Tue, 26 Feb 2013 16:43:48 -0500
Subject: [PATCH] use same images for prev/next as used in image-based
 textbooks

---
 common/static/css/pdfviewer.css       |  5 ++++-
 lms/static/sass/course/_textbook.scss |  1 +
 lms/templates/static_pdfbook.html     | 24 ++++--------------------
 3 files changed, 9 insertions(+), 21 deletions(-)

diff --git a/common/static/css/pdfviewer.css b/common/static/css/pdfviewer.css
index 716db251f0..656bc47c29 100644
--- a/common/static/css/pdfviewer.css
+++ b/common/static/css/pdfviewer.css
@@ -85,13 +85,16 @@ select {
 }
 
 #viewerContainer {
-  overflow: auto;
+/*  overflow: auto; */
   box-shadow: inset 1px 0 0 hsla(0,0%,100%,.05);
 /*  position: absolute;
   top: 32px;
   right: 0;
   bottom: 0;
   left: 0; */
+/* switch to using these instead: */
+  position: relative;
+  overflow: hidden;
 }
 
 .toolbar {
diff --git a/lms/static/sass/course/_textbook.scss b/lms/static/sass/course/_textbook.scss
index 4bda5a46d6..af9c2493fd 100644
--- a/lms/static/sass/course/_textbook.scss
+++ b/lms/static/sass/course/_textbook.scss
@@ -102,6 +102,7 @@ div.book-wrapper {
           position: absolute;
           height: 100%;
           width: flex-grid(2, 8);
+          z-index: 1;
 
           a {
             background-color: rgba(#000, .7);
diff --git a/lms/templates/static_pdfbook.html b/lms/templates/static_pdfbook.html
index 53b018302d..013413b62d 100644
--- a/lms/templates/static_pdfbook.html
+++ b/lms/templates/static_pdfbook.html
@@ -41,11 +41,8 @@
   </script>  
 </%block>
 
-<% use_nav_images = False %>
-
 <%include file="/courseware/course_navigation.html" args="active_page='pdftextbook/{0}'.format(book_index)" />
 
-
     <div id="outerContainer">
       <div id="mainContainer" class="book-wrapper">
 
@@ -53,17 +50,6 @@
           <div id="toolbarContainer">
             <div id="toolbarViewer">
               <div id="toolbarViewerLeft">
-                <div class="splitToolbarButton">
-%if use_nav_images != True:
-                  <button class="toolbarButton pageUp" title="Previous Page" id="previous" tabindex="5">
-                    <span>Previous</span>
-                  </button>
-                  <div class="splitToolbarButtonSeparator"></div>
-                  <button class="toolbarButton pageDown" title="Next Page" id="next" tabindex="6">
-                    <span>Next</span>
-                  </button>
-%endif
-                </div>
                 <label id="pageNumberLabel" class="toolbarLabel" for="pageNumber">Page: </label>
                 <input type="number" id="pageNumber" class="toolbarField pageNumber" value="1" size="4" min="1" tabindex="7">
                 </input>
@@ -116,17 +102,15 @@
               </li>
           </%def>
 
-          <% index = 0 %>
-          % for entry in textbook['chapters']:
-            <% index += 1 %>
-            ${print_entry(entry, index)}
+          % for (index, entry) in enumerate(textbook['chapters']):
+            ${print_entry(entry, index+1)}
           % endfor
       </ul>
     </section>
 %endif
 
         <section id="viewerContainer" class="book">
-%if use_nav_images:
+	  <!-- use same page-turning as used in image-based textbooks -->
           <nav>
             <ul>
               <li class="last">
@@ -137,7 +121,7 @@
               </li>
             </ul>
           </nav>
-%endif
+
           <div id="viewer" contextmenu="viewerContextMenu"></div>
         </div>