Removed onclick calls to js functions in html as they fail to work in firefox.

Tomasz-Kluczkowski · Tomasz-Kluczkowski · commit 3d32d48ef3a9 · 2018-02-10T18:54:04.000Z
Have to replace class declaration as it is not recognised by firefox.
diff --git a/index.html b/index.html
@@ -23,14 +23,14 @@
             </a>
         </div>
 
-        <div class="navigation__button-open-container" onclick="openNav()">
+        <div class="navigation__button-open-container">
             <div class="navigation__hamburger-bar"></div>
             <div class="navigation__hamburger-bar"></div>
             <div class="navigation__hamburger-bar"></div>
         </div>
 
         <div class="navigation__button-close-container">
-            <a href="javascript:void(0)" class="navigation__button-close" onclick="closeNav()">
+            <a href="javascript:void(0)" class="navigation__button-close">
                 <i class="navigation__button-close fas fa-times"></i>
             </a>
         </div>
diff --git a/static/js/site.js b/static/js/site.js
@@ -2,6 +2,8 @@ var buttonOpenContainer = $(".navigation__button-open-container");
 var hamburgerBar = $(".navigation__hamburger-bar");
 var navigationOverlay = $(".navigation__overlay");
 var buttonCloseContainer = $(".navigation__button-close-container");
+var navigationButtonClose = $(".navigation__button-close");
+
 
 function openNav() {
     buttonOpenContainer.css("pointer-events", "none");
@@ -23,6 +25,8 @@ function showCloseButton() {
     buttonCloseContainer.css("display", "inline");
 }
 
+buttonOpenContainer.on("click", openNav);
+navigationButtonClose.on("click", closeNav);
 
 // Responsive background image constructor.
 class ResponsiveBackgroundImage {
