Merge remote-tracking branch 'origin/master'

Author: theangryhobbit

Diff for: assets/js/tabs.js

@@ -1,11 +1,12 @@
-const removeActiveClasses = function (ulElement) {
+function removeActiveClasses(ulElement) {
+
     const lis = ulElement.querySelectorAll('li');
     Array.prototype.forEach.call(lis, function(li) {
         li.classList.remove('active');
     });
 }
 
-const getChildPosition = function (element) {
+function getChildPosition(element) {
     var parent = element.parentNode;
     var i = 0;
     for (var i = 0; i < parent.children.length; i++) {
@@ -24,16 +25,17 @@ window.addEventListener('load', function () {
         link.addEventListener('click', function (event) {
             event.preventDefault();
 
-            liTab = link.parentNode;
-            ulTab = liTab.parentNode;
-            position = getChildPosition(liTab);
+            const liTab = link.parentNode;
+            const ulTab = liTab.parentNode;
+            const position = getChildPosition(liTab);
             if (liTab.className.includes('active')) {
                 return;
             }
 
             removeActiveClasses(ulTab);
-            tabContentId = ulTab.getAttribute('data-tab');
-            tabContentElement = document.getElementById(tabContentId);
+            const tabContentId = ulTab.getAttribute('data-tab');
+            const tabContentElement = document.getElementById(tabContentId);
+
             removeActiveClasses(tabContentElement);
 
             tabContentElement.querySelectorAll('li')[position].classList.add('active');