Merge pull request #73 from joshuaisaact/minor-fixes

fix: flickering on navigation from destination to activity page by pr…
joshuaisaact · Nov 6, 2024 · f501201 · f501201
2 parents 761014b + 83adccf
commit f501201
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/src/pages/DestinationSummary.tsx b/src/pages/DestinationSummary.tsx
@@ -20,17 +20,14 @@ function DestinationSummary() {
   // Load activities if needed when switching to activities or insights
   useEffect(() => {
     const loadActivitiesIfNeeded = async () => {
-      if (
-        (activeTab === 'activities' || activeTab === 'insights') &&
-        destinationName &&
-        !activities[destinationName]
-      ) {
+      if (destinationName && !activities[destinationName]) {
         await loadDestinationActivities(destinationName);
       }
     };
 
     loadActivitiesIfNeeded();
-  }, [activeTab, destinationName, activities, loadDestinationActivities]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, []);
 
   const destination = destinations.find((dest) => dest.destinationName === destinationName);
   const destinationActivities = destinationName ? activities[destinationName] || [] : [];