Remove nesting of hidden sections on sustainer page

This makes the page more similar to how it has worked in previous years.

conservancy/static/js/supporter-page.js

@@ -163,17 +163,6 @@ $(document).ready(function() {
         $expandlink.on('click', function(event) {
             $expandlink.fadeOut('slow');
             $ourexpandablesection.find('.read-more').each(function(index) { $(this).click(); });
-            // Don't laugh. I'm running this twice because the expandable
-            // sections are hidden by actually their text from the DOM and
-            // storing it in a closure, to be reinstated later. That means when
-            // the above ".find('.read-more')" runs it only finds the one
-            // top-level .read-more element because the children have been
-            // temporarily removed from the page. After this top-level element
-            // has been clicked to restore it's body, we can run this again and
-            // .find() the children elements. I suspect this may have worked in
-            // the past if there were multiple sections all at the top-level,
-            // rather than being nested under "Our Year in Review".
-            $ourexpandablesection.find('.read-more').each(function(index) { $(this).click(); });
         });
     });
 });

conservancy/templates/supporter/index.html

@@ -131,6 +131,7 @@ account will be approved.</p>
 <div class="picture-small right">
   <a href="https://sfconservancy.org/videos/sfc-introduction_1080p.mp4"><img src="https://sfconservancy.org/videos/sfc-introduction-video_poster.jpg" alt="Thumbnail of video showing a tree and Software Freedom" /></a>
 </div>
+</div>
 
 
 <h3 id="NewStaff">New staff!</h3>
@@ -284,7 +285,6 @@ reach for reproducibility. </p>
 </div>
 <a class="expander" data-expand-link-text="(Expand All Sections)"></a>
 </div>
-</div>
 
 <div class="donate-sidebar">
   <details>