fix: revert closeOnBlur changes

app/assets/javascripts/components/NotesContextMenu.tsx

@@ -10,7 +10,7 @@ type Props = {
 
 const NotesContextMenu = observer(({ appState }: Props) => {
   const contextMenuRef = useRef<HTMLDivElement>();
-  const [closeOnBlur, blurLocked, setLockCloseOnBlur] = useCloseOnBlur(
+  const [closeOnBlur, setLockCloseOnBlur] = useCloseOnBlur(
     contextMenuRef,
     (open: boolean) => appState.notes.setContextMenuOpen(open)
   );
@@ -37,7 +37,6 @@ const NotesContextMenu = observer(({ appState }: Props) => {
       <NotesOptions
         appState={appState}
         closeOnBlur={closeOnBlur}
-        blurLocked={blurLocked}
         setLockCloseOnBlur={setLockCloseOnBlur}
       />
     </div>

app/assets/javascripts/components/NotesOptions.tsx

@@ -12,18 +12,20 @@ import {
 type Props = {
   appState: AppState;
   closeOnBlur: (event: { relatedTarget: EventTarget | null }) => void;
-  blurLocked: boolean;
   setLockCloseOnBlur: (lock: boolean) => void;
 };
 
 export const NotesOptions = observer(
-  ({ appState, closeOnBlur, blurLocked, setLockCloseOnBlur }: Props) => {
+  ({ appState, closeOnBlur, setLockCloseOnBlur }: Props) => {
     const [tagsMenuOpen, setTagsMenuOpen] = useState(false);
     const [tagsMenuPosition, setTagsMenuPosition] = useState({
       top: 0,
       right: 0,
     });
-    const [shouldTrashNotes, setShouldTrashNotes] = useState(false);
+    const [lockedBlurAction, setLockedBlurAction] = useState<
+      Promise<void> | null
+    >(null);
+    const [shouldRunLockedBlurAction, setShouldRunLockedBlurAction] = useState(false);
 
     const notes = Object.values(appState.notes.selectedNotes);
     const hidePreviews = !notes.some((note) => !note.hidePreview);
@@ -41,23 +43,6 @@ export const NotesOptions = observer(
       'cursor-pointer hover:bg-contrast color-text bg-transparent px-3 ' +
       'text-left';
 
-    useEffect(() => {
-      const openTrashAlert = async () => {
-        if (shouldTrashNotes && blurLocked) {
-          setShouldTrashNotes(false);
-          await appState.notes.setTrashSelectedNotes(!trashed, trashButtonRef);
-          setLockCloseOnBlur(false);
-        }
-      };
-      openTrashAlert();
-    }, [
-      appState.notes,
-      blurLocked,
-      setLockCloseOnBlur,
-      shouldTrashNotes,
-      trashed,
-    ]);
-
     return (
       <>
         <Switch
@@ -186,7 +171,8 @@ export const NotesOptions = observer(
           onBlur={closeOnBlur}
           className={`${buttonClass} py-1.5`}
           onClick={async () => {
-            setShouldTrashNotes(true);
+            setLockCloseOnBlur(true);
+            await appState.notes.setTrashSelectedNotes(!trashed, trashButtonRef);
             setLockCloseOnBlur(true);
           }}
         >

app/assets/javascripts/components/NotesOptionsPanel.tsx

@@ -23,7 +23,7 @@ export const NotesOptionsPanel = observer(({ appState }: Props) => {
   });
   const buttonRef = useRef<HTMLButtonElement>();
   const panelRef = useRef<HTMLDivElement>();
-  const [closeOnBlur, blurLocked, setLockCloseOnBlur] = useCloseOnBlur(panelRef, setOpen);
+  const [closeOnBlur, setLockCloseOnBlur] = useCloseOnBlur(panelRef, setOpen);
 
   return (
     <Disclosure
@@ -71,7 +71,6 @@ export const NotesOptionsPanel = observer(({ appState }: Props) => {
           <NotesOptions
             appState={appState}
             closeOnBlur={closeOnBlur}
-            blurLocked={blurLocked}
             setLockCloseOnBlur={setLockCloseOnBlur}
           />
         )}

app/assets/javascripts/components/utils.ts

@@ -11,7 +11,6 @@ export function useCloseOnBlur(
   setOpen: (open: boolean) => void
 ): [
   (event: { relatedTarget: EventTarget | null }) => void,
-  boolean,
   StateUpdater<boolean>
 ] {
   const [locked, setLocked] = useState(false);
@@ -27,7 +26,6 @@ export function useCloseOnBlur(
       },
       [container, setOpen, locked]
     ),
-    locked,
     setLocked,
   ];
 }