Use a forwardRef-less Row component internally by default (#2640)

* Use a forwardRef-less Row component internally by default

* Pin node in CI

.github/workflows/ci.yml

@@ -11,7 +11,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2
         with:
-          node-version: '16.x'
+          node-version: '16.8'
           check-latest: true
       - uses: actions/cache@v2
         with:

src/Row.tsx

@@ -109,6 +109,8 @@ function Row<R, SR>(
   );
 }
 
-export default memo(forwardRef(Row)) as <R, SR>(
+export default memo(Row) as <R, SR>(props: RowRendererProps<R, SR>) => JSX.Element;
+
+export const RowWithRef = memo(forwardRef(Row)) as <R, SR>(
   props: RowRendererProps<R, SR> & RefAttributes<HTMLDivElement>
 ) => JSX.Element;

src/hooks/useCombinedRefs.ts

@@ -6,7 +6,7 @@ export function useCombinedRefs<T>(...refs: readonly React.Ref<T>[]) {
       for (const ref of refs) {
         if (typeof ref === 'function') {
           ref(handle);
-        } else if (ref !== null) {
+        } else if (ref !== null && 'current' in ref) {
           // @ts-expect-error: https://github.com/DefinitelyTyped/DefinitelyTyped/issues/31065
           ref.current = handle;
         }

src/index.ts

@@ -1,6 +1,6 @@
 export { default } from './DataGrid';
 export type { DataGridProps, DataGridHandle } from './DataGrid';
-export { default as Row } from './Row';
+export { RowWithRef as Row } from './Row';
 export * from './Columns';
 export * from './formatters';
 export { default as TextEditor } from './editors/TextEditor';