fix: avoid re-render after 1st resize (#1669)

dhis2 · Mar 31, 2021 · c77d8e4 · c77d8e4
1 parent e0a34e4
commit c77d8e4
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/packages/plugin/src/ChartPlugin.js b/packages/plugin/src/ChartPlugin.js
@@ -14,6 +14,7 @@ const ChartPlugin = ({
 }) => {
     const canvasRef = useRef(undefined)
     const prevStyle = useRef(style)
+    const prevRenderCounter = useRef(renderCounter)
 
     const renderVisualization = useCallback(
         animation => {
@@ -59,7 +60,10 @@ const ChartPlugin = ({
     }, [visualization, responses, extraOptions])
 
     useEffect(() => {
-        renderCounter !== null && renderVisualization(0)
+        if (renderCounter !== prevRenderCounter.current) {
+            renderVisualization(0)
+            prevRenderCounter.current = renderCounter
+        }
 
         /* eslint-disable-next-line react-hooks/exhaustive-deps */
     }, [renderCounter])