Set max height for config menu

The config menu has gotten out of control recently, but rather than
reducing functionality, I'm just going to set a max height for the div
and allow scrolling within the menu.

Ultimately though this indicates that the app is getting a bit too
complicated (imo). Striking a balance between customization and
minimalism is less of a priority for me nowadays though, hence why I'm
willing to let it slide for now. At some point, maybe when there are
more contributors, it could be nice to refactor this in some way so that
it isn't overwhelming to new users who are looking to customize their
instance (that's just me speculating btw, I haven't actually heard from
anyone who thinks there are too many options in that menu).
main
Ben Busby 2021-11-01 16:55:33 -06:00
parent c766554eea
commit b73c14c7cc
No known key found for this signature in database
GPG Key ID: 339B7B7EB5333D14
3 changed files with 4 additions and 2 deletions

View File

@ -102,6 +102,7 @@ button::-moz-focus-inner {
} }
.open { .open {
overflow-y: scroll;
padding-bottom: 20px; padding-bottom: 20px;
} }

View File

@ -26,7 +26,7 @@ const setupConfigLayout = () => {
if (content.style.maxHeight) { if (content.style.maxHeight) {
content.style.maxHeight = null; content.style.maxHeight = null;
} else { } else {
content.style.maxHeight = content.scrollHeight + "px"; content.style.maxHeight = "400px";
} }
content.classList.toggle("open"); content.classList.toggle("open");

View File

@ -41,9 +41,10 @@
} }
.content { .content {
max-height: 720px; max-height: 400px;
padding: 18px; padding: 18px;
border-radius: 10px; border-radius: 10px;
overflow-y: scroll;
} }
.collapsible { .collapsible {