Issue 5787 - Add Topics Accordion to Article Template on help.eyeo.com

static/js/main.js

 (function(){
   document.addEventListener("DOMContentLoaded", function()
   {
 
     /**************************************************************************
      * General
      **************************************************************************/
 
     // Change html class name from "no-js" to "js"
     document.documentElement.className = "js";
@@ skipping 175 matching lines @@
       }
     }
 
     Accordion.prototype._onClick = function(event)
     {
       if (!event.target.classList.contains("accordion-toggle-button")) return;
       
       this.toggleSection(event.target);
     }
 
-    if (document.getElementById('product-topics-accordion'))

[juliandoucette, 2017/10/11 15:09:48]

NIT: I would query once and [check, use] the result. But the difference is
negligible at this scale.

[ire, 2017/10/11 17:01:41]

I prefer your suggestion. Done.

-    {
-      new Accordion(document.getElementById('product-topics-accordion'));
+    var productTopicsAccordion = document.getElementById('product-topics-accordion');
+    if (productTopicsAccordion)
+    {
+      new Accordion(productTopicsAccordion);
     }
 
   }, false);
 }());