From 1c1686c065830d3800d597fec5e29fd7b4cc09f0 Mon Sep 17 00:00:00 2001
From: XhmikosR <xhmikosr@gmail.com>
Date: Sun, 25 Aug 2019 10:01:18 +0300
Subject: [PATCH] footer.hbs: tweak scrollToTop.

* use `addEventListener`
* use `event.preventDefault()`
* remove unneeded parentheses
* remove `$` from the variable name
---
 layouts/partials/footer.hbs | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/layouts/partials/footer.hbs b/layouts/partials/footer.hbs
index b63bb0f751b54..586d8a8b76ad2 100644
--- a/layouts/partials/footer.hbs
+++ b/layouts/partials/footer.hbs
@@ -29,16 +29,18 @@
 
 <link rel="stylesheet" href="/static/css/prism-tomorrow.css" media="all">
 <script>
-  var $scrollToTop = document.getElementById('scroll-to-top');
+  var scrollToTop = document.getElementById('scroll-to-top');
+
   (window.onscroll = function() {
     window.requestAnimationFrame(function() {
-      $scrollToTop.style.display = (window.pageYOffset > window.innerHeight) ? 'block' : 'none';
+      scrollToTop.style.display = window.pageYOffset > window.innerHeight ? 'block' : 'none';
     })
   })();
-  $scrollToTop.onclick = function() {
+
+  scrollToTop.addEventListener('click', function (e) {
+    e.preventDefault();
     window.scrollTo(0, 0);
-    return false;
-  };
+  });
 </script>
 
 {{!-- Load Modernizr dynamically for IE10/11 and add test classes for sticky footer with flexbox --}}