fix(AnalyticalTable): Resize table content correctly if columns are h…

…idden or minWidth is set (#228)
SAP · Nov 18, 2019 · af1aadc · af1aadc
1 parent fe98dcb
commit af1aadc
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/packages/main/src/components/AnalyticalTable/hooks/useWindowResize.ts b/packages/main/src/components/AnalyticalTable/hooks/useWindowResize.ts
@@ -12,6 +12,12 @@ export const useWindowResize = () => {
 
   const observer = useRef(new MutationObserver(onWindowResize));
 
+  useEffect(() => {
+    if (headerRef.current && headerRef.current.getBoundingClientRect().width !== 0) {
+      setTableWidth(headerRef.current.getBoundingClientRect().width);
+    }
+  }, [headerRef.current, setTableWidth]);
+
   useEffect(() => {
     Device.resize.attachHandler(onWindowResize, null);
     return () => {

diff --git a/packages/main/src/components/AnalyticalTable/virtualization/VirtualTableBody.tsx b/packages/main/src/components/AnalyticalTable/virtualization/VirtualTableBody.tsx
@@ -47,6 +47,9 @@ export const VirtualTableBody = (props) => {
         if (resizedColumns.hasOwnProperty(item.accessor)) {
           return resizedColumns[item.accessor];
         }
+        if (item.hasOwnProperty('show') && !item.show) {
+          return 0;
+        }
         return item.minWidth ? item.minWidth : DEFAULT_COLUMN_WIDTH;
       })
       .reduce((acc, val) => acc + val, 0);