Merge pull request #453 from codomposer/fix/memory_leak_mainchart

Fix: Memory leak in MainChart ColorSchemeChange listener

Author: mrholek

src/views/dashboard/MainChart.js

@@ -7,7 +7,7 @@ const MainChart = () => {
   const chartRef = useRef(null)
 
   useEffect(() => {
-    document.documentElement.addEventListener('ColorSchemeChange', () => {
+    const handleColorSchemeChange = () => {
       if (chartRef.current) {
         setTimeout(() => {
           chartRef.current.options.scales.x.grid.borderColor = getStyle(
@@ -23,7 +23,10 @@ const MainChart = () => {
           chartRef.current.update()
         })
       }
-    })
+    }
+    
+    document.documentElement.addEventListener('ColorSchemeChange', handleColorSchemeChange)
+    return () => document.documentElement.removeEventListener('ColorSchemeChange', handleColorSchemeChange)
   }, [chartRef])
 
   const random = () => Math.round(Math.random() * 100)