Fix

harshit078 · Nov 15, 2024 · 1669612 · 1669612
1 parent cfe6c01
commit 1669612
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 13 deletions.
diff --git a/packages/twenty-front/src/modules/action-menu/components/RecordIndexActionMenuBar.tsx b/packages/twenty-front/src/modules/action-menu/components/RecordIndexActionMenuBar.tsx
@@ -34,10 +34,6 @@ export const RecordIndexActionMenuBar = () => {
 
   const pinnedEntries = actionMenuEntries.filter((entry) => entry.isPinned);
 
-  if (contextStoreNumberOfSelectedRecords === 0) {
-    return null;
-  }
-
   return (
     <BottomBar
       bottomBarId={getActionBarIdFromActionMenuId(actionMenuId)}

diff --git a/packages/twenty-front/src/modules/action-menu/components/RecordIndexActionMenuEffect.tsx b/packages/twenty-front/src/modules/action-menu/components/RecordIndexActionMenuEffect.tsx
@@ -42,22 +42,20 @@ export const RecordIndexActionMenuEffect = () => {
 
   const isCommandMenuOpened = useRecoilValue(isCommandMenuOpenedState);
 
+  useEffect(() => {
+    if (contextStoreNumberOfSelectedRecords === 0) {
+      closeActionBar();
+    }
+  }, [contextStoreNumberOfSelectedRecords, closeActionBar]);
+
   useEffect(() => {
     if (contextStoreNumberOfSelectedRecords > 0 && !isDropdownOpen) {
       // We only handle opening the ActionMenuBar here, not the Dropdown.
       // The Dropdown is already managed by sync handlers for events like
       // right-click to open and click outside to close.
       openActionBar();
     }
-    if (contextStoreNumberOfSelectedRecords === 0 && isDropdownOpen) {
-      closeActionBar();
-    }
-  }, [
-    contextStoreNumberOfSelectedRecords,
-    openActionBar,
-    closeActionBar,
-    isDropdownOpen,
-  ]);
+  }, [contextStoreNumberOfSelectedRecords, openActionBar, isDropdownOpen]);
 
   useEffect(() => {
     if (isRightDrawerOpen || isCommandMenuOpened) {