enhance: cleanup, ux/ui changes, safer approach to android's ptr

components/pull-to-refresh.js

@@ -2,12 +2,10 @@ import { useRouter } from 'next/router'
 import { useState, useRef, useEffect, useCallback } from 'react'
 import styles from './pull-to-refresh.module.css'
 
-const PULL_THRESHOLD = 300
-const REFRESH_TIMEOUT = 500
+const REFRESH_THRESHOLD = 300
 
 export default function PullToRefresh ({ children, android }) {
   const router = useRouter()
-  const [isRefreshing, setIsRefreshing] = useState(false)
   const [pullDistance, setPullDistance] = useState(0)
   const [isPWA, setIsPWA] = useState(false)
   const [isAndroid, setIsAndroid] = useState(false)
@@ -21,9 +19,7 @@ export default function PullToRefresh ({ children, android }) {
     setIsPWA(androidPWA || iosPWA)
   }
 
-  useEffect(() => {
-    checkPWA()
-  }, [])
+  useEffect(checkPWA, [])
 
   const handleTouchStart = useCallback((e) => {
     // don't handle if the user is not scrolling from the top of the page, is not on a PWA or if we want Android's native PTR
@@ -32,19 +28,15 @@ export default function PullToRefresh ({ children, android }) {
   }, [isPWA, isAndroid, android])
 
   const handleTouchMove = useCallback((e) => {
-    if (touchStartY.current === 0) return
+    if (touchStartY.current === 0) return // prevent unintended refresh by checking if the user has actually started touching
     touchEndY.current = e.touches[0].clientY
     setPullDistance(touchEndY.current - touchStartY.current)
   }, [])
 
   const handleTouchEnd = useCallback(() => {
-    if (touchStartY.current === 0 || touchEndY.current === 0) return
-    if (touchEndY.current - touchStartY.current > PULL_THRESHOLD) {
-      setIsRefreshing(true)
+    if (touchStartY.current === 0 || touchEndY.current === 0) return // if the user has started touch or is actually 'pulling'
+    if (touchEndY.current - touchStartY.current > REFRESH_THRESHOLD) {
       router.push(router.asPath) // reload the same path
-      setTimeout(() => {
-        setIsRefreshing(false)
-      }, REFRESH_TIMEOUT) // simulate loading time
     }
     setPullDistance(0) // using this to reset the message behavior
     touchStartY.current = 0 // avoid random refreshes by resetting touch
@@ -54,10 +46,12 @@ export default function PullToRefresh ({ children, android }) {
   useEffect(() => {
     // don't handle if the user is not on a PWA or if we want Android's native PTR
     if (!isPWA || (isAndroid && !android)) return
+    document.body.style.overscrollBehaviorY = 'contain' // disable Android's native PTR
     document.addEventListener('touchstart', handleTouchStart)
     document.addEventListener('touchmove', handleTouchMove)
     document.addEventListener('touchend', handleTouchEnd)
     return () => {
+      document.body.style.overscrollBehaviorY = '' // if unmounted reset the overscroll behavior
       document.removeEventListener('touchstart', handleTouchStart)
       document.removeEventListener('touchmove', handleTouchMove)
       document.removeEventListener('touchend', handleTouchEnd)
@@ -65,29 +59,21 @@ export default function PullToRefresh ({ children, android }) {
   }, [isPWA, isAndroid, android, handleTouchStart, handleTouchMove, handleTouchEnd])
 
   const getPullMessage = () => {
-    if (isRefreshing) return 'refreshing...'
-    if (pullDistance > PULL_THRESHOLD) return 'release to refresh'
+    if (pullDistance > REFRESH_THRESHOLD) return 'release to refresh'
     if (pullDistance > 0) return 'pull down to refresh'
     return ''
   }
 
   return (
-    <div className={android ? styles.pullToRefreshContainer : ''}> {/* android prop if true disables its native PTR */}
+    <div>
       <div
         onTouchStart={handleTouchStart}
         onTouchMove={handleTouchMove}
         onTouchEnd={handleTouchEnd}
       >
-        {pullDistance > 0 || isRefreshing
-          ? (
-            <>
-              <p className={`${styles.pullMessage} ${pullDistance > 50 || isRefreshing ? styles.fadeIn : ''}`}>
-                {getPullMessage()}
-              </p>
-              {isRefreshing && <div className={styles.spacer} />}
-            </>
-            )
-          : null}
+        <p className={`${styles.pullMessage} ${pullDistance > 50 && styles.fadeIn}`}>
+          {getPullMessage()}
+        </p>
         {children}
       </div>
     </div>

components/pull-to-refresh.module.css

@@ -1,13 +1,9 @@
-.pullToRefreshContainer {
-    overscroll-behavior: contain;
-}
-
 .pullMessage {
     position: fixed;
     left: 50%;
     top: -50px;
     transform: translateX(-50%);
-    font-size: 18px;
+    font-size: small;
     color: #a5a5a5;
     opacity: 0;
     transition: opacity 0.3s ease-in-out;
@@ -18,8 +14,3 @@
     opacity: 1;
     top: 0;
 }
-
-.spacer {
-    position: relative;
-    margin-bottom: 32px;
-}