fix: firefox/safari to redirect to /explore in HomeLayout (#3001)

pull/3040/head
Zhiyuan Zheng 1 year ago committed by GitHub
parent 8d08cfe1c9
commit 51a7934616
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

@ -1,6 +1,6 @@
import { Button, IconButton, Tooltip } from "@mui/joy"; import { Button, IconButton, Tooltip } from "@mui/joy";
import classNames from "classnames"; import classNames from "classnames";
import { Suspense } from "react"; import { Suspense, useEffect } from "react";
import { Outlet, useLocation } from "react-router-dom"; import { Outlet, useLocation } from "react-router-dom";
import useLocalStorage from "react-use/lib/useLocalStorage"; import useLocalStorage from "react-use/lib/useLocalStorage";
import Icon from "@/components/Icon"; import Icon from "@/components/Icon";
@ -19,15 +19,16 @@ const HomeLayout = () => {
const [collapsed, setCollapsed] = useLocalStorage<boolean>("navigation-collapsed", false); const [collapsed, setCollapsed] = useLocalStorage<boolean>("navigation-collapsed", false);
// Redirect to explore page if not logged in. // Redirect to explore page if not logged in.
if ( useEffect(() => {
!currentUser && if (
([Routes.HOME, Routes.TIMELINE, Routes.RESOURCES, Routes.INBOX, Routes.ARCHIVED, Routes.SETTING] as string[]).includes( !currentUser &&
location.pathname, ([Routes.HOME, Routes.TIMELINE, Routes.RESOURCES, Routes.INBOX, Routes.ARCHIVED, Routes.SETTING] as string[]).includes(
) location.pathname,
) { )
navigateTo("/explore"); ) {
return; navigateTo("/explore");
} }
}, []);
return ( return (
<div className="w-full min-h-full"> <div className="w-full min-h-full">

Loading…
Cancel
Save