fix: Set width of columns without mutations

material-table-core · Jun 1, 2021 · 4e08c89 · 4e08c89
1 parent 390fb59
commit 4e08c89
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 13 deletions.
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -1 +1,2 @@
-*       @mbrn
+*       @domino051
+*       @oze4
diff --git a/src/utils/data-manager.js b/src/utils/data-manager.js
@@ -63,18 +63,6 @@ export default class DataManager {
   }
 
   setColumns(columns) {
-    const undefinedWidthColumns = columns.filter((c) => {
-      if (c.hidden) {
-        // Hidden column
-        return false;
-      }
-      if (c.columnDef && c.columnDef.tableData && c.columnDef.tableData.width) {
-        // tableData.width already calculated
-        return false;
-      }
-      // Calculate width if no value provided
-      return c.width === undefined;
-    });
     let usedWidth = ['0px'];
 
     this.columns = columns.map((columnDef, index) => {
@@ -105,6 +93,18 @@ export default class DataManager {
 
       return { ...columnDef, tableData };
     });
+    const undefinedWidthColumns = this.columns.filter((c) => {
+      if (c.hidden) {
+        // Hidden column
+        return false;
+      }
+      if (c.columnDef && c.columnDef.tableData && c.columnDef.tableData.width) {
+        // tableData.width already calculated
+        return false;
+      }
+      // Calculate width if no value provided
+      return c.width === undefined;
+    });
 
     usedWidth = '(' + usedWidth.join(' + ') + ')';
     undefinedWidthColumns.forEach((columnDef) => {