fix: missing loading animation

gouravmpk · Sep 21, 2022 · 0b2b27e · 0b2b27e
1 parent 3dbb2f7
commit 0b2b27e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 11 deletions.
diff --git a/src/components/common/Loader/index.tsx b/src/components/common/Loader/index.tsx
@@ -3,7 +3,6 @@ import ClipLoader from "react-spinners/ClipLoader";
 import { Flex } from "~/components/common/Flex";
 import { Text } from "~/components/common/Text";
 import { colors } from "~/utils/colors";
-
 const messages = [
   "Searching Podcast Index",
   "Searching YouTube",
@@ -18,16 +17,17 @@ export const Loader = () => {
   const [msgIndex, setMsgIndex] = useState(0);
 
   useEffect(() => {
-    const messageTimeout = setTimeout(doIndex, 5000);
+    if (msgIndex === messages.length - 1) {
+      return;
+    }
 
-    return function cleanup() {
-      clearTimeout(messageTimeout);
-    };
-  }, [msgIndex]);
+    const messageTimeout = setTimeout(
+      () => setMsgIndex((index) => (index + 1) % messages.length),
+      1000
+    );
 
-  function doIndex() {
-    if (msgIndex < messages.length - 1) setMsgIndex(msgIndex + 1);
-  }
+    return () => clearTimeout(messageTimeout);
+  }, [msgIndex]);
 
   return (
     <Flex grow={1} align="center" justify="center">

diff --git a/src/stores/useDataStore/index.ts b/src/stores/useDataStore/index.ts
@@ -13,7 +13,6 @@ type DataStore = {
   selectedNode: NodeExtended | null;
   selectedTimestamp: NodeExtended | null;
   setCameraAnimation: (cameraAnimation: gsap.core.Tween | null) => void;
-  setLoadingData: (bool: boolean) => void;
   fetchData: (search?: string | null) => void;
   setSelectedNode: (selectedNode: NodeExtended | null) => void;
   setSelectedTimestamp: (selectedTimestamp: NodeExtended | null) => void;
@@ -29,7 +28,6 @@ export const useDataStore = create<DataStore>((set, get) => ({
   selectedNode: null,
   selectedTimestamp: null,
   setCameraAnimation: (cameraAnimation) => set({ cameraAnimation }),
-  setLoadingData: (isFetching) => set({ isFetching }),
   setHoveredNode: (hoveredNode) => set({ hoveredNode }),
   setSelectedNode: (selectedNode) =>
     set({ selectedNode, isTimestampLoaded: false }),