Merge pull request #459 from PeterYurkovich/ou-721

OU-721: reset showgraph state when unmounting button

Author: openshift-merge-bot[bot]

web/src/components/MetricsPage.tsx

@@ -326,6 +326,17 @@ export const ToggleGraph: React.FC = () => {
 
   const dispatch = useDispatch();
   const toggle = React.useCallback(() => dispatch(toggleGraphs()), [dispatch]);
+  // Use an empty useEffect to get access to the cleanup function so that if graphs are
+  // currently hidden then we toggle one more time as we unmount
+  React.useEffect(
+    () => () => {
+      if (hideGraphs) {
+        toggle();
+      }
+    },
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    [hideGraphs],
+  );
 
   const icon = hideGraphs ? <ChartLineIcon /> : <CompressIcon />;