Allow classic jump list to scroll and fix div tag

Updated to match the fixed version from the parallel layout. Also discovered a `</div>` tag was missing. Fixes jashkenas#302.
Truffula · Oct 22, 2014 · 555b750 · 555b750
1 parent 33e21f0
commit 555b750
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 15 deletions.
diff --git a/resources/classic/docco.css b/resources/classic/docco.css
@@ -105,9 +105,18 @@ ul.sections > li > div {
   display: block;
 }
 
+#jump_page_wrapper{
+  position: fixed;
+  right: 0;
+  top: 0;
+  bottom: 0;
+}
+
 #jump_page {
   padding: 5px 0 3px;
   margin: 0 0 25px 25px;
+  max-height: 100%;
+  overflow: auto;
 }
 
 #jump_page .source {

diff --git a/resources/classic/docco.jst b/resources/classic/docco.jst
@@ -16,13 +16,16 @@
           <a class="large" href="javascript:void(0);">Jump To &hellip;</a>
           <a class="small" href="javascript:void(0);">+</a>
           <div id="jump_wrapper">
-          <div id="jump_page">
-            <% for (var i=0, l=sources.length; i<l; i++) { %>
-              <% var source = sources[i]; %>
-              <a class="source" href="<%= path.basename(destination(source)) %>">
-                <%= path.basename(source) %>
-              </a>
-            <% } %>
+            <div id="jump_page_wrapper">
+              <div id="jump_page">
+                <% for (var i=0, l=sources.length; i<l; i++) { %>
+                  <% var source = sources[i]; %>
+                  <a class="source" href="<%= path.basename(destination(source)) %>">
+                    <%= path.basename(source) %>
+                  </a>
+                <% } %>
+              </div>
+            </div>
           </div>
         </li>
       </ul>

diff --git a/resources/parallel/docco.jst b/resources/parallel/docco.jst
@@ -16,14 +16,15 @@
           <a class="large" href="javascript:void(0);">Jump To &hellip;</a>
           <a class="small" href="javascript:void(0);">+</a>
           <div id="jump_wrapper">
-          <div id="jump_page_wrapper">
-            <div id="jump_page">
-              <% for (var i=0, l=sources.length; i<l; i++) { %>
-                <% var source = sources[i]; %>
-                <a class="source" href="<%= path.basename(destination(source)) %>">
-                  <%= path.basename(source) %>
-                </a>
-              <% } %>
+            <div id="jump_page_wrapper">
+              <div id="jump_page">
+                <% for (var i=0, l=sources.length; i<l; i++) { %>
+                  <% var source = sources[i]; %>
+                  <a class="source" href="<%= path.basename(destination(source)) %>">
+                    <%= path.basename(source) %>
+                  </a>
+                <% } %>
+              </div>
             </div>
           </div>
         </li>