fix: preserve header pinning order

Fixes #3915
TanStack · May 15, 2022 · ade3cc7 · ade3cc7 · vercel · May 15, 2022
1 parent 4c57a8a
commit ade3cc7
Showing 1 changed file with 11 additions and 6 deletions.
diff --git a/packages/table-core/src/features/Headers.ts b/packages/table-core/src/features/Headers.ts
@@ -121,15 +121,20 @@ export const Headers = {
           instance.getState().columnPinning.right,
         ],
         (allColumns, leafColumns, left, right) => {
-          const leftColumns = leafColumns.filter(column =>
-            left?.includes(column.id)
-          )
-          const rightColumns = leafColumns.filter(column =>
-            right?.includes(column.id)
-          )
+          const leftColumns =
+            left
+              ?.map(columnId => leafColumns.find(d => d.id === columnId)!)
+              .filter(Boolean) ?? []
+
+          const rightColumns =
+            right
+              ?.map(columnId => leafColumns.find(d => d.id === columnId)!)
+              .filter(Boolean) ?? []
+
           const centerColumns = leafColumns.filter(
             column => !left?.includes(column.id) && !right?.includes(column.id)
           )
+
           const headerGroups = buildHeaderGroups(
             allColumns,
             [...leftColumns, ...centerColumns, ...rightColumns],