From c826ad66df066b90b09009f2f4b83b56d018436e Mon Sep 17 00:00:00 2001
From: Armand Philippot <git@armandphilippot.com>
Date: Wed, 15 Nov 2023 16:37:16 +0100
Subject: refactor(hooks): rewrite useScrollPosition hook

* return the scroll position (both X and Y)
* no longer accepts arguments
* add tests
---
 src/components/templates/layout/layout.tsx | 24 ++++++++++--------------
 1 file changed, 10 insertions(+), 14 deletions(-)

(limited to 'src/components')

diff --git a/src/components/templates/layout/layout.tsx b/src/components/templates/layout/layout.tsx
index c7b8dbd..5dad8a2 100644
--- a/src/components/templates/layout/layout.tsx
+++ b/src/components/templates/layout/layout.tsx
@@ -7,7 +7,6 @@ import {
   type ReactElement,
   type ReactNode,
   useRef,
-  useState,
   type CSSProperties,
   type FormEvent,
   useCallback,
@@ -367,19 +366,16 @@ export const Layout: FC<LayoutProps> = ({
     subjectOf: { '@id': `${url}` },
   };
 
-  const [backToTopClassName, setBackToTopClassName] = useState<string>(
-    `${styles['back-to-top']} ${styles['back-to-top--hidden']}`
-  );
-  const updateBackToTopClassName = () => {
-    const visibleBreakpoint = 300;
-    setBackToTopClassName(
-      window.scrollY > visibleBreakpoint
-        ? `${styles['back-to-top']} ${styles['back-to-top--visible']}`
-        : `${styles['back-to-top']} ${styles['back-to-top--hidden']}`
-    );
-  };
-
-  useScrollPosition(updateBackToTopClassName);
+  const scrollPos = useScrollPosition();
+  const backToTopBreakpoint = 300;
+  const backToTopClassName = [
+    styles['back-to-top'],
+    styles[
+      scrollPos.y > backToTopBreakpoint
+        ? 'back-to-top--visible'
+        : 'back-to-top--hidden'
+    ],
+  ].join(' ');
 
   const topRef = useRef<HTMLSpanElement>(null);
   const giveFocusToTopRef = () => {
-- 
cgit v1.2.3