feat: Multiple minor changes (#769)

app/assets/javascripts/components/NotesListOptionsMenu.tsx

@@ -2,11 +2,11 @@ import { WebApplication } from '@/ui_models/application';
 import { CollectionSort, PrefKey } from '@standardnotes/snjs';
 import { observer } from 'mobx-react-lite';
 import { FunctionComponent } from 'preact';
-import { useState } from 'preact/hooks';
+import { useRef, useState } from 'preact/hooks';
 import { Icon } from './Icon';
 import { Menu } from './menu/Menu';
 import { MenuItem, MenuItemSeparator, MenuItemType } from './menu/MenuItem';
-import { toDirective } from './utils';
+import { toDirective, useCloseOnClickOutside } from './utils';
 
 type Props = {
   application: WebApplication;
@@ -108,8 +108,16 @@ flex flex-col py-2 bottom-0 left-2 absolute';
       application.setPreference(PrefKey.NotesHideProtected, !hideProtected);
     };
 
+    const menuRef = useRef<HTMLDivElement>(null);
+
+    useCloseOnClickOutside(menuRef as any, (open: boolean) => {
+      if (!open) {
+        setShowMenuFalse();
+      }
+    });
+
     return (
-      <div className={menuClassName}>
+      <div ref={menuRef} className={menuClassName}>
         <Menu a11yLabel="Sort by" closeMenu={setShowMenuFalse}>
           <div className="px-3 my-1 text-xs font-semibold color-text uppercase">
             Sort by

app/assets/javascripts/components/QuickSettingsMenu/QuickSettingsMenu.tsx

@@ -76,7 +76,6 @@ const QuickSettingsMenu: FunctionComponent<MenuProps> = observer(
     }, [focusModeEnabled]);
 
     const reloadThemes = useCallback(() => {
-      application.streamItems(ContentType.Theme, () => {
       const themes = application.getDisplayableItems(
         ContentType.Theme
       ) as SNTheme[];
@@ -100,31 +99,44 @@ const QuickSettingsMenu: FunctionComponent<MenuProps> = observer(
       setDefaultThemeOn(
         !themes.find((theme) => theme.active && !theme.isLayerable())
       );
-      });
     }, [application]);
 
     const reloadToggleableComponents = useCallback(() => {
-      application.streamItems(ContentType.Component, () => {
       const toggleableComponents = (
-          application.getDisplayableItems(
+        application.getDisplayableItems(ContentType.Component) as SNComponent[]
-            ContentType.Component
-          ) as SNComponent[]
       ).filter((component) =>
         [ComponentArea.EditorStack, ComponentArea.TagsList].includes(
           component.area
         )
       );
       setToggleableComponents(toggleableComponents);
-      });
     }, [application]);
 
     useEffect(() => {
+      const cleanupItemStream = application.streamItems(
+        ContentType.Theme,
+        () => {
           reloadThemes();
-    }, [reloadThemes]);
+        }
+      );
+
+      return () => {
+        cleanupItemStream();
+      };
+    }, [application, reloadThemes]);
 
     useEffect(() => {
+      const cleanupItemStream = application.streamItems(
+        ContentType.Component,
+        () => {
           reloadToggleableComponents();
-    }, [reloadToggleableComponents]);
+        }
+      );
+
+      return () => {
+        cleanupItemStream();
+      };
+    }, [application, reloadToggleableComponents]);
 
     useEffect(() => {
       if (themesMenuOpen) {
@@ -274,10 +286,9 @@ const QuickSettingsMenu: FunctionComponent<MenuProps> = observer(
             </Disclosure>
           ) : null}
           {toggleableComponents.map((component) => (
-            <Switch
+            <button
-              className="sn-dropdown-item focus:bg-info-backdrop focus:shadow-none"
+              className="sn-dropdown-item justify-between focus:bg-info-backdrop focus:shadow-none"
-              checked={component.active}
+              onClick={() => {
-              onChange={() => {
                 toggleComponent(component);
               }}
             >
@@ -285,7 +296,8 @@ const QuickSettingsMenu: FunctionComponent<MenuProps> = observer(
                 <Icon type="window" className="color-neutral mr-2" />
                 {component.name}
               </div>
-            </Switch>
+              <Switch checked={component.active} className="px-0" />
+            </button>
           ))}
           <FocusModeSwitch
             application={application}