fix: avoid stale scroll retry sync (#1495)

Author: zombieJ

src/Table.tsx

@@ -465,11 +465,11 @@ const Table = <RecordType extends DefaultRecordType>(
 
       // Delay to force scroll position if not sync
       // ref: https://github.com/ant-design/ant-design/issues/37179
-      if (target.scrollLeft !== scrollLeft) {
-        setTimeout(() => {
+      setTimeout(() => {
+        if (target.scrollLeft !== scrollLeft) {
           target.scrollLeft = scrollLeft;
-        }, 0);
-      }
+        }
+      }, 0);
     }
   }
 

tests/Table.spec.jsx

@@ -1322,4 +1322,47 @@ describe('Table.Basic', () => {
     fireEvent.scroll(tableBody);
     expect(onScroll).toHaveBeenCalled();
   });
+
+  it('does not retry stale scrollLeft sync', () => {
+    vi.useFakeTimers();
+
+    try {
+      const { container } = render(
+        createTable({
+          scroll: { x: 100, y: 100 },
+        }),
+      );
+
+      const tableHeader = container.querySelector('.rc-table-header');
+      const tableBody = container.querySelector('.rc-table-body');
+      let headerScrollLeft = 0;
+      let bodyScrollLeft = 0;
+
+      Object.defineProperty(tableHeader, 'scrollLeft', {
+        get() {
+          return headerScrollLeft;
+        },
+        set(value) {
+          headerScrollLeft = Math.trunc(value);
+        },
+      });
+      Object.defineProperty(tableBody, 'scrollLeft', {
+        get() {
+          return bodyScrollLeft;
+        },
+      });
+
+      bodyScrollLeft = 794.171875;
+      fireEvent.scroll(tableBody);
+
+      bodyScrollLeft = 0;
+      fireEvent.scroll(tableBody);
+
+      vi.runAllTimers();
+
+      expect(headerScrollLeft).toBe(0);
+    } finally {
+      vi.useRealTimers();
+    }
+  });
 });