Issue 5406 - Create Site Footer Component for Help Center

static/js/main.js

 (function(){
-  document.addEventListener("DOMContentLoaded", function() {

[juliandoucette, 2017/08/22 14:33:42]

NIT: We are supposed to put brackets on the next line :( 

(We could bring this up in a frontend dev meeting because I think it makes sense
for these to be on the same line.)

[ire, 2017/09/04 20:28:06]

Done.

+  document.addEventListener("DOMContentLoaded", function()
+  {
 
     // Change html class name from "no-js" to "js"
     document.documentElement.className = "js";
 
+    // Toggle Navbar Collapse
+    function toggleNavbarCollapse()
+    {
+      var navbarCollapseEls = this.parentElement.getElementsByClassName("navbar-collapse");
+      for (var i = 0; i < navbarCollapseEls.length; i++)
+      {
+        navbarCollapseEls[i]
+          .classList.toggle("open")
+      }
+    }
+
+    var toggleNavbarCollapseEls = document.getElementsByClassName("toggle-navbar-collapse");
+    for (var i = 0; i < toggleNavbarCollapseEls.length; i++)
+    {
+      toggleNavbarCollapseEls[i]
+        .addEventListener("click", toggleNavbarCollapse, false);
+    }
+
     // Custom Select
-    var customSelects = document.getElementsByClassName('custom-select-selected');
-    function onClickCustomSelect(e) {
-      var options = e.target.nextElementSibling;
+    function onClickCustomSelect()
+    {
+      var options = this.nextElementSibling;
       if (options.getAttribute("aria-hidden") == "true")
       {
         options.removeAttribute("aria-hidden");
-        e.target.setAttribute("aria-expanded", "true");
-      } 
+        this.setAttribute("aria-expanded", "true");
+      }
       else
       {
         options.setAttribute("aria-hidden", "true");
-        e.target.setAttribute("aria-expanded", "false");
+        this.setAttribute("aria-expanded", "false");
       }
     }
-    for (var i = 0; i < customSelects.length; i++) {
-      customSelects[i].addEventListener("click", onClickCustomSelect, false);

[juliandoucette, 2017/08/22 14:33:41]

NIT: We could handle this event on the body instead.

[ire, 2017/09/04 20:28:06]

Agreed. I would like to change the way both this and the event listener on the
navbarCollapse work so I'll create a separate issue for this

[juliandoucette, 2017/09/06 17:48:19]

Acknowledged.

+
+    var customSelectEls = document.getElementsByClassName('custom-select-selected');
+    for (var i = 0; i < customSelectEls.length; i++)
+    {
+      customSelectEls[i]
+        .addEventListener("click", onClickCustomSelect, false);
+      customSelectEls[i]
+        .nextElementSibling
+        .setAttribute("aria-hidden", "true");
     }
- 
+
   }, false);
 }());