fix: missing memo dependencies and css

apotdevin · Aug 1, 2023 · 2859f29 · 2859f29
1 parent e362890
commit 2859f29
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/client/src/components/sankey/index.tsx b/src/client/src/components/sankey/index.tsx
@@ -86,7 +86,7 @@ export const Sankey = ({ data, width, height }: SankeyProps) => {
  },
  ],
  };
- }, [data, themeContext]);
+ }, [data, themeContext, height]);
 
  return (
  <ReactEChartsCore

diff --git a/src/client/src/views/forwards/forwardSankey.tsx b/src/client/src/views/forwards/forwardSankey.tsx
@@ -11,7 +11,7 @@ const SANKEY_HEIGHT_DESKTOP = '800px';
 
 const Wrapper = styled.div`
  height: ${SANKEY_HEIGHT_DESKTOP};
- maxheight: ${SANKEY_HEIGHT_DESKTOP};
+ max-height: ${SANKEY_HEIGHT_DESKTOP};
  width: 100%;
 
  @media (${mediaWidths.mobile}) {
@@ -50,7 +50,7 @@ export const ForwardSankey = ({ days }: { days: number }) => {
  });
 
  return orderedData;
- }, [data]);
+ }, [data, loading]);
 
  if (loading || !data?.getForwards?.length) {
  return null;