From 303ad7ff7b7190b85f1c41fd5ad1a87f9d802c3c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=B6rn=20Nettingsmeier?= <nettings@stackingdwarves.net>
Date: Mon, 3 Feb 2014 02:45:21 +0100
Subject: [PATCH] simplify tree generation, restore broken autoscrolling (could
 use some tweaking)

---
 _manual/17_mixing/02_panning.html             |  4 +--
 .../02_panning/02_balance_control.html        |  7 ++++--
 _plugins/manual.rb                            | 25 +++++++++++++------
 source/css/app.css                            |  4 ---
 4 files changed, 24 insertions(+), 16 deletions(-)

diff --git a/_manual/17_mixing/02_panning.html b/_manual/17_mixing/02_panning.html
index 41a2360..5af2467 100644
--- a/_manual/17_mixing/02_panning.html
+++ b/_manual/17_mixing/02_panning.html
@@ -41,8 +41,8 @@ title: Panning
   of these situations, but in the future will offer more.
 </p>
 <p>
-  In addition to the panners, Ardour has a stereo balance control for subtle
-  corrections to an existing stereo image.
+  In addition to the panners, Ardour has a balance control for subtle
+  corrections to existing stereo images.
 </p>
 
 {% children %}
diff --git a/_manual/17_mixing/02_panning/02_balance_control.html b/_manual/17_mixing/02_panning/02_balance_control.html
index 0821b1b..3c2f838 100644
--- a/_manual/17_mixing/02_panning/02_balance_control.html
+++ b/_manual/17_mixing/02_panning/02_balance_control.html
@@ -8,12 +8,15 @@ title: Balance Control
   traditional <strong>balance control</strong> by right-clicking on the
   panner widget.
 </p>
+<img class="left" src="/images/stereo-balance.png" alt="Stereo Balance
+control"/>
 <p>
   When the balance is centered, the incoming signals will be unaffected.
   Moving it to one side will linearly attenuate the signal of the opposite
   side.
 </p>
 <div class="well">
-  While it is considerably less flexible than the stereo panner, it work
-  with arbitrary content without the danger of comb filter artefacts.
+  While the balance control is considerably less flexible than the stereo 
+  panner, it works with arbitrary content without danger of introducing
+  comb filter artefacts.
 </div>
diff --git a/_plugins/manual.rb b/_plugins/manual.rb
index b3d8dad..24a532c 100644
--- a/_plugins/manual.rb
+++ b/_plugins/manual.rb
@@ -165,9 +165,9 @@ module Manual
       position ? [current, position, level + 1] : [current]
     end
 
-    def sanitize_NMTOKEN(s)
-      'ID'+s.gsub(/[^0-9A-z:_.-]/, '_')
-    end
+#    def sanitize_NMTOKEN(s)
+#      'ID'+s.gsub(/[^0-9A-z:_.-]/, '_')
+#    end
     
     def render(context)
 
@@ -181,14 +181,14 @@ module Manual
       current_a = current.split('/').reject(&:empty?)
 
       tree = Manual.traverse_data(@@data_tree) do |entry|
-
+      
           url = entry[:url]
 
           url_a = url.split('/').reject(&:empty?)
 
           depth = url_a.length
           is_current, position, level = *process_hierarchy(current_a, url_a)
-
+          
           # this massively speeds up build time by not including the whole menu tree for each page
           next if depth > 1 && current_a[0] != url_a[0]
 
@@ -205,7 +205,7 @@ module Manual
 
               erb = ::ERB.new <<-HTML
                   <dt class="<%= css_classes %>">
-                      <a id="<%= sanitize_NMTOKEN(entry[:url]) %>" href="<%= entry[:url] %>"><%= entry[:menu_title] %></a>
+                      <a href="<%= entry[:url] %>"><%= entry[:menu_title] %></a>
                   </dt>
                   <dd class="<%= css_classes %>">
                       <% if entry[:children].any? %>
@@ -225,7 +225,7 @@ module Manual
 
                 erb = ::ERB.new <<-HTML
                     <dt class="<%= css_classes %>">
-                        <a id="<%= sanitize_NMTOKEN(entry[:url]) %>" href="<%= entry[:url] %>"><%= entry[:menu_title] %></a>
+                        <a href="<%= entry[:url] %>"><%= entry[:menu_title] %></a>
                     </dt>
                     <dd class="<%= css_classes %>">
                     </dd>
@@ -234,9 +234,18 @@ module Manual
                 erb.result(binding)
              end
           end
+          
+         
       end
 
-      "<dl>#{tree.join}</dl>"
+      "<dl>#{tree.join}</dl>
+      <script type='text/javascript'><!--
+        offset = document.getElementsByClassName('active')[0].offsetTop;
+        height = document.getElementById('tree').clientHeight;
+        if (offset > (height * .7)) {
+          tree.scrollTop = offset - height * .3;
+        }
+      --></script>"
 
     end
 
diff --git a/source/css/app.css b/source/css/app.css
index ad851c6..e2972dc 100644
--- a/source/css/app.css
+++ b/source/css/app.css
@@ -271,10 +271,6 @@
     white-space:nowrap;
 }
 
-#content img {
-    margin: 1em 0 1em 0;
-
-}
 #content td {
     padding: 0 1em 0 1em;
 }