hq/Bpistle
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: ensure titlebar appears when scrolling to top on mobile

Browse source
This commit is contained in:
codytseng 2024-12-08 15:25:08 +08:00
parent d6f5a67004
commit 49a9824034
2 changed files with 18 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/renderer/src/layouts/PrimaryPageLayout/index.tsx
View file

@ -32,13 +32,18 @@ const PrimaryPageLayout = forwardRef(({ children }: { children?: React.ReactNode
const handleScroll = () => { const handleScroll = () => {
const scrollTop = scrollAreaRef.current?.scrollTop || 0 const scrollTop = scrollAreaRef.current?.scrollTop || 0
const diff = scrollTop - lastScrollTop const diff = scrollTop - lastScrollTop
if (diff > 50) { if (scrollTop <= 100) {
setVisible(false)
setLastScrollTop(scrollTop)
} else if (diff < -50) {
setVisible(true) setVisible(true)
setLastScrollTop(scrollTop) setLastScrollTop(scrollTop)
return
} }
if (diff > 50) {
setVisible(false)
} else if (diff < -50) {
setVisible(true)
}
setLastScrollTop(scrollTop)
} }
const scrollArea = scrollAreaRef.current const scrollArea = scrollAreaRef.current

11
src/renderer/src/layouts/SecondaryPageLayout/index.tsx
View file

@ -26,9 +26,16 @@ export default function SecondaryPageLayout({
useEffect(() => { useEffect(() => {
const handleScroll = () => { const handleScroll = () => {
const scrollTop = scrollAreaRef.current?.scrollTop || 0 const scrollTop = scrollAreaRef.current?.scrollTop || 0
if (scrollTop > lastScrollTop) { const diff = scrollTop - lastScrollTop
if (scrollTop <= 100) {
setVisible(true)
setLastScrollTop(scrollTop)
return
}
if (diff > 50) {
setVisible(false) setVisible(false)
} else { } else if (diff < -50) {
setVisible(true) setVisible(true)
} }
setLastScrollTop(scrollTop) setLastScrollTop(scrollTop)