fix: useAppVisible hook

pengx17 · May 25, 2021 · 0f3ad46 · 0f3ad46
1 parent 7757e0e
commit 0f3ad46
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/src/App.tsx b/src/App.tsx
@@ -2,9 +2,16 @@ import React, { useRef, useState } from "react";
 
 const useAppVisible = () => {
   const [visible, setVisible] = useState(logseq.isMainUIVisible);
-  logseq.on("ui:visible:changed", async ({ visible }) => {
-    setVisible(visible);
-  });
+  React.useEffect(() => {
+    const eventName = "ui:visible:changed";
+    const handler = async ({ visible }: any) => {
+      setVisible(visible);
+    };
+    logseq.on(eventName, handler);
+    return () => {
+      logseq.off(eventName, handler);
+    };
+  }, []);
   return visible;
 };