From 56878f647ea0f1066fa3e222d7aa0d83057f496d Mon Sep 17 00:00:00 2001
From: Armand Philippot <git@armandphilippot.com>
Date: Mon, 13 Nov 2023 17:45:59 +0100
Subject: refactor(components): rewrite PostsList component

* remove NoResults component and move logic to Search page
* add a usePostsList hook
* remove Pagination from PostsList (it is only used if javascript is
disabled and not on every posts list)
* replace `byYear` prop with `sortByYear`
* replace `loadMore` prop with `onLoadMore`
* remove `showLoadMoreBtn` (we can use `loadMore` prop instead to
determine if we need to display the button)
* replace `titleLevel` prop with `headingLvl`
* add `firstNewResult` prop to handle focus on the new results when
loading more article (we should not focus a useless span but the item
directly)
---
 src/utils/helpers/index.ts      |  1 +
 src/utils/helpers/refs.test.tsx | 28 ++++++++++++++++++++++++++++
 src/utils/helpers/refs.ts       | 16 ++++++++++++++++
 3 files changed, 45 insertions(+)
 create mode 100644 src/utils/helpers/refs.test.tsx
 create mode 100644 src/utils/helpers/refs.ts

(limited to 'src/utils/helpers')

diff --git a/src/utils/helpers/index.ts b/src/utils/helpers/index.ts
index f340a49..79077de 100644
--- a/src/utils/helpers/index.ts
+++ b/src/utils/helpers/index.ts
@@ -2,6 +2,7 @@ export * from './author';
 export * from './images';
 export * from './pages';
 export * from './reading-time';
+export * from './refs';
 export * from './rss';
 export * from './schema-org';
 export * from './strings';
diff --git a/src/utils/helpers/refs.test.tsx b/src/utils/helpers/refs.test.tsx
new file mode 100644
index 0000000..93e5f89
--- /dev/null
+++ b/src/utils/helpers/refs.test.tsx
@@ -0,0 +1,28 @@
+import { describe, it, jest } from '@jest/globals';
+import { render } from '@testing-library/react';
+import { forwardRef, useImperativeHandle } from 'react';
+import { mergeRefs } from './refs';
+
+const refValue = 'minus architecto qui';
+const TestComponentWithForwardedRef = forwardRef((_, ref) => {
+  useImperativeHandle(ref, () => refValue);
+  return null;
+});
+TestComponentWithForwardedRef.displayName = 'TestComponentWithForwardedRef';
+
+describe('merge-refs', () => {
+  it('can merge a ref function with a ref object', () => {
+    const refFn = jest.fn();
+    const refObj = { current: null };
+
+    const TestComponent = () => (
+      <TestComponentWithForwardedRef ref={mergeRefs([refFn, refObj])} />
+    );
+
+    render(<TestComponent />);
+
+    expect(refFn).toHaveBeenCalledTimes(1);
+    expect(refFn).toHaveBeenLastCalledWith(refValue);
+    expect(refObj.current).toBe(refValue);
+  });
+});
diff --git a/src/utils/helpers/refs.ts b/src/utils/helpers/refs.ts
new file mode 100644
index 0000000..74a695a
--- /dev/null
+++ b/src/utils/helpers/refs.ts
@@ -0,0 +1,16 @@
+import type { LegacyRef, MutableRefObject, RefCallback } from 'react';
+import type { Nullable } from '../../types';
+
+export const mergeRefs =
+  <T = unknown>(
+    refs: (MutableRefObject<T> | LegacyRef<T> | undefined | null)[]
+  ): RefCallback<T> =>
+  (value) => {
+    refs.forEach((ref) => {
+      if (typeof ref === 'function') {
+        ref(value);
+      } else if (ref !== null) {
+        (ref as MutableRefObject<Nullable<T>>).current = value;
+      }
+    });
+  };
-- 
cgit v1.2.3