Merge branch 'main' of github.com:Alexandrebdry/react-use-scroll-navigate

Author: Alexandrebdry

.gitignore

@@ -1,3 +1,7 @@
+# built files
+cjs/
+esm/
+
 # Logs
 logs
 *.log
@@ -51,8 +55,8 @@ build/Release
 node_modules/
 jspm_packages/
 
-# TypeScript v1 declaration files
-typings/
+# Snowpack dependency directory (https://snowpack.dev/)
+web_modules/
 
 # TypeScript cache
 *.tsbuildinfo
@@ -94,7 +98,7 @@ dist
 
 # Gatsby files
 .cache/
-# Comment in the public line in if your project uses Gatsby and *not* Next.js
+# Comment in the public line in if your project uses Gatsby and not Next.js
 # https://nextjs.org/blog/next-9-1#public-directory-support
 # public
 
@@ -112,3 +116,6 @@ dist
 
 # TernJS port file
 .tern-port
+
+# Stores VSCode versions used for testing VSCode extensions
+.vscode-test

package-lock.json

@@ -34,7 +34,7 @@ export class ScrollNavigateTopError extends Error {
 export function useScrollNavigate(path = "/", top = 0 , left = 0  ) {
 
     const navigate = useNavigate();
-    const [scrollNavigateError, setScrollNavigateError] = useState() ;
+    const [scrollNavigateError, setScrollNavigateError] = useState(null) ;
     const scrollNavigate = useCallback( (path, top, left) => {
        if (typeof path != "string") {
            setScrollNavigateError(new ScrollNavigatePathError("Path is not a string")) ;
@@ -71,7 +71,7 @@ export function useScrollNavigate(path = "/", top = 0 , left = 0  ) {
 export const NavigateContext = createContext();
 export const NavigateContextProvider = ({children}) => {
     const navigate = useNavigate();
-    const [navigationBeacon, setNavigationBeacon] = useState(false) ;
+    const [navigationBeacon, setNavigationBeacon] = useState(null) ;
     const navigationRef = useRef() ;
 
     const navigateToRef = useCallback((path) => {
@@ -81,10 +81,10 @@ export const NavigateContextProvider = ({children}) => {
     },[navigate, setNavigationBeacon]);
 
     const value = useMemo(() => {navigationRef, navigateToRef},
-        [navigationRef,navigateToRef()]);
+        [navigationRef,navigateToRef]);
 
     useLayoutEffect(() => {
-       if (navigationBeacon === false )return
+       if (navigationBeacon === false || navigationBeacon === null )return
        if(navigationRef.current instanceof HTMLElement) {
            const {left,top} = navigationRef.current.getBoundingClientRect();
            window.scroll({left,top,behavior:'smooth'});