From 90f6d988271ef68a83079feebafcb8bb10af8a11 Mon Sep 17 00:00:00 2001
From: Matthew Gamble <git@matthewgamble.net>
Date: Tue, 7 Mar 2017 09:26:51 +1100
Subject: [PATCH] Don't use event delegation, use aria-expanded for "is
 collapsed" check

---
 tsstats/templates/index.jinja2 | 31 ++++++++++++++++---------------
 1 file changed, 16 insertions(+), 15 deletions(-)

diff --git a/tsstats/templates/index.jinja2 b/tsstats/templates/index.jinja2
index 977d9b1..2293605 100644
--- a/tsstats/templates/index.jinja2
+++ b/tsstats/templates/index.jinja2
@@ -61,21 +61,22 @@
 </div>
 
 <script type="text/javascript">
-  document.addEventListener('click', function(event) {
-    var toggler = event.target;
-    if (toggler.dataset.toggle !== 'collapse') {
-      return;
-    }
-    var toggleTarget = document.getElementById(toggler.dataset.target);
-
-    if (toggleTarget.classList.contains('collapse')) {
-      toggleTarget.classList.remove('collapse');
-      toggler.setAttribute('aria-expanded', false);
-    } else {
-      toggleTarget.classList.add('collapse');
-      toggler.setAttribute('aria-expanded', true);
-    }
-  });
+  (function() {
+    var collapseTogglers = document.querySelectorAll('[data-toggle="collapse"]');
+    var toggleTarget;
+    Array.prototype.forEach.call(collapseTogglers, function(toggler) {
+      toggler.addEventListener('click', function(event) {
+        toggleTarget = document.getElementById(toggler.dataset.target);
+        if (toggler.getAttribute('aria-expanded') === 'true') {
+          toggleTarget.classList.add('collapse');
+          toggler.setAttribute('aria-expanded', false);
+        } else {
+          toggleTarget.classList.remove('collapse');
+          toggler.setAttribute('aria-expanded', true);
+        }
+      });
+    });
+  })();
 </script>
 </body>
 </html>