Commit 6131d007 authored by Lynn's avatar Lynn Committed by GitHub

fix: fix scroll on page navigation (#3893)

* fix: fix scroll on page navigation

* fix: scroll to top when navigating in whole app

* fix: refactor to use useHistory hook

* fix: change scrollToTop component to hook
Co-authored-by: default avatarLynn Yu <lynn.yu@uniswap.org>
parent e6814994
...@@ -2,6 +2,8 @@ import Loader from 'components/Loader' ...@@ -2,6 +2,8 @@ import Loader from 'components/Loader'
import TopLevelModals from 'components/TopLevelModals' import TopLevelModals from 'components/TopLevelModals'
import ApeModeQueryParamReader from 'hooks/useApeModeQueryParamReader' import ApeModeQueryParamReader from 'hooks/useApeModeQueryParamReader'
import { lazy, Suspense } from 'react' import { lazy, Suspense } from 'react'
import { useEffect } from 'react'
import { useHistory } from 'react-router-dom'
import { Redirect, Route, Switch } from 'react-router-dom' import { Redirect, Route, Switch } from 'react-router-dom'
import styled from 'styled-components/macro' import styled from 'styled-components/macro'
...@@ -64,6 +66,17 @@ const Marginer = styled.div` ...@@ -64,6 +66,17 @@ const Marginer = styled.div`
` `
export default function App() { export default function App() {
const history = useHistory()
useEffect(() => {
const unlisten = history.listen(() => {
window.scrollTo(0, 0)
})
return () => {
unlisten()
}
}, [history])
return ( return (
<ErrorBoundary> <ErrorBoundary>
<Route component={GoogleAnalyticsReporter} /> <Route component={GoogleAnalyticsReporter} />
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment