From cbe1feb2a2dd5ef679a8389c0307ebf39ab120bd Mon Sep 17 00:00:00 2001
From: hippoz <paperplanelive@gmail.com>
Date: Sat, 23 Oct 2021 17:49:40 +0300
Subject: [PATCH] fix: prevent UserButton from fetching user info even before
 click

---
 src/components/channel/ChannelCreateButton.js | 4 ++--
 src/components/user/UserButton.js             | 6 ++++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/components/channel/ChannelCreateButton.js b/src/components/channel/ChannelCreateButton.js
index fe80dc3..b3f5bec 100644
--- a/src/components/channel/ChannelCreateButton.js
+++ b/src/components/channel/ChannelCreateButton.js
@@ -80,8 +80,8 @@ export default function ChannelCreateButton() {
         {/* ugly: AnimatePresence is needed to animate the modal closing */}
         <AnimatePresence>
             {(isDialogOpen) && <Modal
-                width="500px"
-                height="300px"
+                width="425px"
+                height="275px"
                 title="Create a channel"
                 alignItems="center"
                 onClose={ handleClose }
diff --git a/src/components/user/UserButton.js b/src/components/user/UserButton.js
index 75bae00..407ca5f 100644
--- a/src/components/user/UserButton.js
+++ b/src/components/user/UserButton.js
@@ -6,10 +6,12 @@ import Modal from "../Modal";
 import { AnimatePresence } from "framer-motion";
 
 export default function ChannelUserButton({ user, subtext }) {
-    const [ isPromptOpen, setIsPromptOpen ] = useState(false);
+    const [isPromptOpen, setIsPromptOpen] = useState(false);
     const [userObject, setUserObject] = useState(null);
 
     useEffect(() => {
+        if (!isPromptOpen) return;
+        if (userObject) return;
         authenticated(`/api/v1/users/user/${user._id}/info`, {
             method: 'GET',
             headers: {
@@ -20,7 +22,7 @@ export default function ChannelUserButton({ user, subtext }) {
                 setUserObject(json.user);
             }
         });
-    }, [user]);
+    }, [user, isPromptOpen, userObject]);
 
     return (
         <>