[Glitch] Fix edge case where settings/admin page sidebar would be incorrectly hidden

Port 00b45b967e to glitch-soc Signed-off-by: Claire <claire.github-309c@sitedethib.com>
2022-02-16 21:44:19 +01:00
parent 41d52ee4b5
commit d70e4e3da0
3 changed files with 6 additions and 14 deletions
--- a/app/javascript/flavours/glitch/packs/settings.js
+++ b/app/javascript/flavours/glitch/packs/settings.js
@@ -7,13 +7,7 @@ function main() {
  const { delegate } = require('@rails/ujs');

  delegate(document, '.sidebar__toggle__icon', 'click', () => {
-    const target = document.querySelector('.sidebar ul');
-
-    if (target.style.display === 'block') {
-      target.style.display = 'none';
-    } else {
-      target.style.display = 'block';
-    }
+    document.querySelector('.sidebar ul').classList.toggle('visible');
  });
 }