Fix edge case where settings/admin page sidebar would be incorrectly hidden (#17580)
This commit is contained in:
parent
1abf0f9000
commit
00b45b967e
|
@ -276,13 +276,7 @@ function main() {
|
||||||
});
|
});
|
||||||
|
|
||||||
delegate(document, '.sidebar__toggle__icon', 'click', () => {
|
delegate(document, '.sidebar__toggle__icon', 'click', () => {
|
||||||
const target = document.querySelector('.sidebar ul');
|
document.querySelector('.sidebar ul').classList.toggle('visible');
|
||||||
|
|
||||||
if (target.style.display === 'block') {
|
|
||||||
target.style.display = 'none';
|
|
||||||
} else {
|
|
||||||
target.style.display = 'block';
|
|
||||||
}
|
|
||||||
});
|
});
|
||||||
|
|
||||||
// Empty the honeypot fields in JS in case something like an extension
|
// Empty the honeypot fields in JS in case something like an extension
|
||||||
|
|
|
@ -322,6 +322,10 @@ $content-width: 840px;
|
||||||
|
|
||||||
& > ul {
|
& > ul {
|
||||||
display: none;
|
display: none;
|
||||||
|
|
||||||
|
&.visible {
|
||||||
|
display: block;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
ul a,
|
ul a,
|
||||||
|
|
Loading…
Reference in a new issue