From 3be87084287109cb8097a58a778d06307a6ecf09 Mon Sep 17 00:00:00 2001
From: Christoph Ladurner <christoph.ladurner@tugraz.at>
Date: Mon, 15 Jun 2020 21:44:38 +0200
Subject: [PATCH] theme.js refactoring

the function hide has been refactored to toggleVisibility. The target element id
is now given by a parameter.
---
 .../semantic-ui/js/invenio_theme_tugraz/theme.js    | 13 +++++--------
 .../templates/invenio_theme_tugraz/header.html      |  2 +-
 2 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/invenio_theme_tugraz/assets/semantic-ui/js/invenio_theme_tugraz/theme.js b/invenio_theme_tugraz/assets/semantic-ui/js/invenio_theme_tugraz/theme.js
index 4068a46..55ba290 100644
--- a/invenio_theme_tugraz/assets/semantic-ui/js/invenio_theme_tugraz/theme.js
+++ b/invenio_theme_tugraz/assets/semantic-ui/js/invenio_theme_tugraz/theme.js
@@ -1,11 +1,8 @@
-export function hide() {
-    var x = document.getElementById('heading');
+export function toggleVisibility(id) {
+    var element = document.getElementById(id);
+    var isHided = element.style.display === "none";
 
-    if (x.style.display === "none") {
-        x.style.display = "block";
-    } else {
-        x.style.display = "none";
-    }
+    element.style.display = isHided ? "block" : "none";
 }
 
-window.hide = hide;
+window.toggleVisibility = toggleVisibility;
diff --git a/invenio_theme_tugraz/templates/invenio_theme_tugraz/header.html b/invenio_theme_tugraz/templates/invenio_theme_tugraz/header.html
index 9b5d632..ce8fa5f 100644
--- a/invenio_theme_tugraz/templates/invenio_theme_tugraz/header.html
+++ b/invenio_theme_tugraz/templates/invenio_theme_tugraz/header.html
@@ -125,7 +125,7 @@
           </div>
 
           <div class="short-menu-right">
-            <div class="short-menu-right-h" onclick="hide();">
+            <div class="short-menu-right-h" onclick="toggleVisibility('heading');">
 
               <i class="fa fa-bars fa-lg"></i><span style="margin-left:3px;">Menu</span>