From d21f8f0633ea505f328a0d7337391b03add5b17a Mon Sep 17 00:00:00 2001 From: sethburkart123 Date: Wed, 19 Jun 2024 22:09:49 +1000 Subject: [PATCH] perf: hide unused elements for better DOM performance --- src/SEQTA.ts | 9 +-------- src/css/injected/sidebar-animation.scss | 4 ++++ 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/src/SEQTA.ts b/src/SEQTA.ts index b2b1561e..022dc7d0 100644 --- a/src/SEQTA.ts +++ b/src/SEQTA.ts @@ -700,13 +700,7 @@ export function tryLoad() { updateIframesWithDarkMode() // Waits for page to call on load, run scripts - document.addEventListener( - 'load', - function () { - removeThemeTagsFromNotices() - }, - true, - ) + removeThemeTagsFromNotices() } function ChangeMenuItemPositions(storage: any) { @@ -795,7 +789,6 @@ function main() { tryLoad() } else { handleDisabled() - window.addEventListener('load', handleDisabled) } } diff --git a/src/css/injected/sidebar-animation.scss b/src/css/injected/sidebar-animation.scss index a6ec6efd..bdbde4c9 100644 --- a/src/css/injected/sidebar-animation.scss +++ b/src/css/injected/sidebar-animation.scss @@ -16,6 +16,10 @@ transform: translateX(-320px); } +#menu li.hasChildren:not(.active) li .sub { + display: none; // improves performance +} + .sub .sub { top: 0 !important; padding-top: 2px;