From 406abb4929c2349167fa0593747cfc0f82898156 Mon Sep 17 00:00:00 2001 From: Aman Harwara Date: Sun, 23 Jul 2023 15:43:02 +0530 Subject: [PATCH] chore: fix links container toggle logic --- .../LinkedItems/LinkedItemBubblesContainer.tsx | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/packages/web/src/javascripts/Components/LinkedItems/LinkedItemBubblesContainer.tsx b/packages/web/src/javascripts/Components/LinkedItems/LinkedItemBubblesContainer.tsx index 6f3809703..6b7cc305d 100644 --- a/packages/web/src/javascripts/Components/LinkedItems/LinkedItemBubblesContainer.tsx +++ b/packages/web/src/javascripts/Components/LinkedItems/LinkedItemBubblesContainer.tsx @@ -119,10 +119,13 @@ const LinkedItemBubblesContainer = ({ ) } - const [isCollapsed, setIsCollapsed] = useState(() => isCollapsedByDefault) - const itemsToDisplay = allItemsLinkedToItem.concat(notesLinkingToItem).concat(filesLinkingToItem) - const visibleItems = isCollapsed ? itemsToDisplay.slice(0, 5) : itemsToDisplay + const ItemsToShowWhenCollapsed = 5 + const [isCollapsed, setIsCollapsed] = useState( + itemsToDisplay.length < ItemsToShowWhenCollapsed ? false : isCollapsedByDefault, + ) + + const visibleItems = isCollapsed ? itemsToDisplay.slice(0, ItemsToShowWhenCollapsed) : itemsToDisplay const nonVisibleItems = itemsToDisplay.length - visibleItems.length const [canShowContainerToggle, setCanShowContainerToggle] = useState(true)