From b1d4ccc27e397f1bb50bf00e5d95afcb891ca2f6 Mon Sep 17 00:00:00 2001
From: Abdullah Atta <abdullahatta@streetwriters.co>
Date: Sat, 18 Mar 2023 12:27:11 +0500
Subject: [PATCH] editor: fix dropdown needing 2 clicks to open

---
 .../editor/src/toolbar/components/dropdown.tsx     | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/packages/editor/src/toolbar/components/dropdown.tsx b/packages/editor/src/toolbar/components/dropdown.tsx
index c8405b599..da7134b6a 100644
--- a/packages/editor/src/toolbar/components/dropdown.tsx
+++ b/packages/editor/src/toolbar/components/dropdown.tsx
@@ -45,7 +45,7 @@ export function Dropdown(props: DropdownProps) {
   const isMobile = useIsMobile();
   const isBottom = toolbarLocation === "bottom";
 
-  const { closePopup, isPopupOpen } = usePopupHandler({
+  const { isPopupOpen } = usePopupHandler({
     group,
     id,
     isOpen,
@@ -82,7 +82,15 @@ export function Dropdown(props: DropdownProps) {
           selectedItem
         )}
         <Icon
-          path={isBottom ? Icons.chevronUp : Icons.chevronDown}
+          path={
+            isBottom
+              ? isPopupOpen
+                ? Icons.chevronDown
+                : Icons.chevronUp
+              : isPopupOpen
+              ? Icons.chevronUp
+              : Icons.chevronDown
+          }
           size={"small"}
           color={"text"}
         />
@@ -91,7 +99,7 @@ export function Dropdown(props: DropdownProps) {
       <MenuPresenter
         isOpen={isPopupOpen}
         items={items}
-        onClose={() => closePopup(id)}
+        onClose={() => setIsOpen(false)}
         position={{
           target: isBottom
             ? getToolbarElement()