From: Claire Date: Wed, 16 Feb 2022 20:44:19 +0000 (+0100) Subject: Fix edge case where settings/admin page sidebar would be incorrectly hidden (#17580) X-Git-Url: https://git.xn--scling-oua.cat.family/?a=commitdiff_plain;h=00b45b967e0c92714e1ec54a2d5c924f8b1dd38b;p=mastodon.git Fix edge case where settings/admin page sidebar would be incorrectly hidden (#17580) --- diff --git a/app/javascript/packs/public.js b/app/javascript/packs/public.js index c0c088646..3d0a937e1 100644 --- a/app/javascript/packs/public.js +++ b/app/javascript/packs/public.js @@ -276,13 +276,7 @@ function main() { }); 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'); }); // Empty the honeypot fields in JS in case something like an extension diff --git a/app/javascript/styles/mastodon/admin.scss b/app/javascript/styles/mastodon/admin.scss index f5741bd50..1921eb146 100644 --- a/app/javascript/styles/mastodon/admin.scss +++ b/app/javascript/styles/mastodon/admin.scss @@ -322,6 +322,10 @@ $content-width: 840px; & > ul { display: none; + + &.visible { + display: block; + } } ul a,