From 37c9f0f2820f75e980f12f0d07231aed69d537b7 2023-11-23 03:41:27
From: Ben Sturmfels
Date: 2023-11-23 03:41:27
Subject: [PATCH] Remove nesting of hidden sections on sustainer page
This makes the page more similar to how it has worked in previous years.
---
diff --git a/conservancy/static/js/supporter-page.js b/conservancy/static/js/supporter-page.js
index 1ae36a46db197226f2f0aec73bb32e64936206e5..76dc07d65685e8916b31efc5cbced4200e3a6908 100644
--- a/conservancy/static/js/supporter-page.js
+++ b/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(); });
});
});
});
diff --git a/conservancy/templates/supporter/index.html b/conservancy/templates/supporter/index.html
index d8605f2e28afa60945ec739f821d516aef015a06..bee59e9338730cd479a88da77cd5f454a77a9687 100644
--- a/conservancy/templates/supporter/index.html
+++ b/conservancy/templates/supporter/index.html
@@ -131,6 +131,7 @@ account will be approved.
+
New staff!
@@ -284,7 +285,6 @@ reach for reproducibility.
-