Merge pull request #1 from mechie/fix-mobile-menu

Use optional chaining to add listeners to optional UI elements.
tchartron · Jan 30, 2022 · 19ce007 · 19ce007
2 parents bfa42d2 + 4fdce5f
commit 19ce007
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/src/js/lang.js b/src/js/lang.js
@@ -1,5 +1,5 @@
 document.addEventListener("DOMContentLoaded", function() {
-    document.getElementById('switch-lang').addEventListener('click', switchLang);
+    document.getElementById('switch-lang')?.addEventListener('click', switchLang);
 })
 
 function switchLang(event) {

diff --git a/src/js/main.js b/src/js/main.js
@@ -23,13 +23,13 @@ document.addEventListener("DOMContentLoaded", function() {
     document.getElementById("syntax_highlight").href = "/syntax-light.css";
   }
   // Switch theme action
-  document.getElementById('switch-theme').addEventListener('click', switchTheme);
+  document.getElementById('switch-theme')?.addEventListener('click', switchTheme);
 
   // ---------------- Toggle Sidebar -------------------------
-  document.getElementById('toggle-sidebar').addEventListener('click', toggleSidebar);
+  document.getElementById('toggle-sidebar')?.addEventListener('click', toggleSidebar);
 
   // ---------------- Toggle Mobile menu -------------------------
-  document.getElementById('toggle-mobile-menu').addEventListener('click', toggleMobileMenu);
+  document.getElementById('toggle-mobile-menu')?.addEventListener('click', toggleMobileMenu);
 });
 
 function switchTheme() {

diff --git a/static/js/lang.js b/static/js/lang.js
diff --git a/static/js/main.js b/static/js/main.js