From 51a7934616341c49671ade9bad71f703132911c5 Mon Sep 17 00:00:00 2001 From: Zhiyuan Zheng Date: Mon, 26 Feb 2024 13:38:24 +0800 Subject: [PATCH] fix: firefox/safari to redirect to /explore in HomeLayout (#3001) --- web/src/layouts/HomeLayout.tsx | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/web/src/layouts/HomeLayout.tsx b/web/src/layouts/HomeLayout.tsx index 36344694..2bf63c93 100644 --- a/web/src/layouts/HomeLayout.tsx +++ b/web/src/layouts/HomeLayout.tsx @@ -1,6 +1,6 @@ import { Button, IconButton, Tooltip } from "@mui/joy"; import classNames from "classnames"; -import { Suspense } from "react"; +import { Suspense, useEffect } from "react"; import { Outlet, useLocation } from "react-router-dom"; import useLocalStorage from "react-use/lib/useLocalStorage"; import Icon from "@/components/Icon"; @@ -19,15 +19,16 @@ const HomeLayout = () => { const [collapsed, setCollapsed] = useLocalStorage("navigation-collapsed", false); // Redirect to explore page if not logged in. - if ( - !currentUser && - ([Routes.HOME, Routes.TIMELINE, Routes.RESOURCES, Routes.INBOX, Routes.ARCHIVED, Routes.SETTING] as string[]).includes( - location.pathname, - ) - ) { - navigateTo("/explore"); - return; - } + useEffect(() => { + if ( + !currentUser && + ([Routes.HOME, Routes.TIMELINE, Routes.RESOURCES, Routes.INBOX, Routes.ARCHIVED, Routes.SETTING] as string[]).includes( + location.pathname, + ) + ) { + navigateTo("/explore"); + } + }, []); return (