From e0cce43511bde23db0cbf90f4dbf9343960ec835 Mon Sep 17 00:00:00 2001
From: MaheshtheDev <mahesh.svmr@gmail.com>
Date: Thu, 18 Jul 2024 22:20:03 -0700
Subject: [PATCH] chore: Refactor handlePopoverChange function

---
 apps/extension/content/ContentApp.tsx | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/apps/extension/content/ContentApp.tsx b/apps/extension/content/ContentApp.tsx
index 217f72ff..de760407 100644
--- a/apps/extension/content/ContentApp.tsx
+++ b/apps/extension/content/ContentApp.tsx
@@ -164,13 +164,6 @@ export default function ContentApp({
     }
   };
 
-  async function handlePopoverChange() {
-    setIsPopoverOpen(!isPopoverOpen);
-    if (isPopoverOpen) {
-      await getSpaces();
-    }
-  }
-
   async function sendUrlToAPI(spaces: string[]) {
     setLoading(true);
 
@@ -248,10 +241,13 @@ export default function ContentApp({
     <div className="flex justify-end items-end min-h-screen h-full w-full">
       <Toaster />
 
-      <Popover open={isPopoverOpen} onOpenChange={handlePopoverChange}>
+      <Popover
+        open={isPopoverOpen}
+        onOpenChange={() => setIsPopoverOpen(!isPopoverOpen)}
+      >
         <TooltipProvider>
           <Tooltip>
-            <TooltipTrigger asChild>
+            <TooltipTrigger onClick={async () => await getSpaces()} asChild>
               <PopoverTrigger
                 className={`${hover || isPopoverOpen ? "opacity-100" : "opacity-75 pointer-events-none translate-x-3/4"} focus-within:translate-x-0 focus-visible:translate-x-0 size-12 hover:bg-black p-2 rounded-l-2xl transition bg-secondary border-2 border-border opacity-0 absolute flex bottom-20 items-center text-lg`}
               >