changed breakdown charts memo to only re-render if the chart is open

growthepie · Nov 26, 2024 · 796bac4 · 796bac4
1 parent 648c9bb
commit 796bac4
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/components/layout/Economics/BreakdownCharts.tsx b/components/layout/Economics/BreakdownCharts.tsx
@@ -1153,7 +1153,12 @@ function BreakdownCharts({
 }
 
 export default React.memo(BreakdownCharts, (prevProps, nextProps) => {
-  // Custom comparison function to prevent unnecessary re-renders
+  // Prevent re-renders if isOpen is false in both prev and next props
+  if (!prevProps.isOpen && !nextProps.isOpen) {
+    return true; // No need to re-render
+  }
+
+  // Normal comparison logic
   return (
     prevProps.data === nextProps.data &&
     prevProps.dailyData === nextProps.dailyData &&
@@ -1164,3 +1169,4 @@ export default React.memo(BreakdownCharts, (prevProps, nextProps) => {
     prevProps.isMonthly === nextProps.isMonthly
   );
 });
+