From 926dce37f814faa2de84c5fb6aa024e1bb83dc3c Mon Sep 17 00:00:00 2001
From: Manav Rathi <manav@mrmr.io>
Date: Thu, 28 Nov 2024 16:52:13 +0530
Subject: [PATCH] Extract

---
 web/apps/photos/src/pages/_app.tsx            | 18 ++----------
 .../photos/components/utils/use-is-offline.ts | 28 +++++++++++++++++++
 2 files changed, 31 insertions(+), 15 deletions(-)
 create mode 100644 web/packages/new/photos/components/utils/use-is-offline.ts

diff --git a/web/apps/photos/src/pages/_app.tsx b/web/apps/photos/src/pages/_app.tsx
index 4e8cdceadf..ab1b816649 100644
--- a/web/apps/photos/src/pages/_app.tsx
+++ b/web/apps/photos/src/pages/_app.tsx
@@ -19,6 +19,7 @@ import {
     updateAvailableForDownloadDialogAttributes,
     updateReadyToInstallDialogAttributes,
 } from "@/new/photos/components/utils/download";
+import { useIsOffline } from "@/new/photos/components/utils/use-is-offline";
 import { useLoadingBar } from "@/new/photos/components/utils/use-loading-bar";
 import { photosDialogZIndex } from "@/new/photos/components/utils/z-index";
 import { runMigrations } from "@/new/photos/services/migrations";
@@ -54,9 +55,6 @@ export default function App({ Component, pageProps }: AppProps) {
     const router = useRouter();
     const [isI18nReady, setIsI18nReady] = useState<boolean>(false);
     const [loading, setLoading] = useState(false);
-    const [offline, setOffline] = useState(
-        typeof window !== "undefined" && !window.navigator.onLine,
-    );
     const [showNavbar, setShowNavBar] = useState(false);
     const [watchFolderView, setWatchFolderView] = useState(false);
     const [watchFolderFiles, setWatchFolderFiles] = useState<FileList>(null);
@@ -65,6 +63,7 @@ export default function App({ Component, pageProps }: AppProps) {
     const [notificationAttributes, setNotificationAttributes] =
         useState<NotificationAttributes>(null);
 
+    const isOffline = useIsOffline();
     const { showMiniDialog, miniDialogProps } = useAttributedMiniDialog();
     const { loadingBarRef, showLoadingBar, hideLoadingBar } = useLoadingBar();
     const [themeColor, setThemeColor] = useLocalState(
@@ -127,9 +126,6 @@ export default function App({ Component, pageProps }: AppProps) {
         if (isDesktop) void resumeExportsIfNeeded();
     }, []);
 
-    const setUserOnline = () => setOffline(false);
-    const setUserOffline = () => setOffline(true);
-
     useEffect(() => {
         const query = new URLSearchParams(window.location.search);
         const needsFamilyRedirect = query.get("redirect") == "families";
@@ -154,14 +150,6 @@ export default function App({ Component, pageProps }: AppProps) {
         router.events.on("routeChangeComplete", () => {
             setLoading(false);
         });
-
-        window.addEventListener("online", setUserOnline);
-        window.addEventListener("offline", setUserOffline);
-
-        return () => {
-            window.removeEventListener("online", setUserOnline);
-            window.removeEventListener("offline", setUserOffline);
-        };
     }, []);
 
     useEffect(() => {
@@ -210,7 +198,7 @@ export default function App({ Component, pageProps }: AppProps) {
                 <CssBaseline enableColorScheme />
                 {showNavbar && <AppNavbar />}
                 <OfflineMessageContainer>
-                    {isI18nReady && offline && t("OFFLINE_MSG")}
+                    {isI18nReady && isOffline && t("OFFLINE_MSG")}
                 </OfflineMessageContainer>
                 <LoadingBar color="#51cd7c" ref={loadingBarRef} />
 
diff --git a/web/packages/new/photos/components/utils/use-is-offline.ts b/web/packages/new/photos/components/utils/use-is-offline.ts
new file mode 100644
index 0000000000..a0519364f3
--- /dev/null
+++ b/web/packages/new/photos/components/utils/use-is-offline.ts
@@ -0,0 +1,28 @@
+import { useEffect, useState } from "react";
+
+/**
+ * A hook that returns true if the browser is currently offline.
+ *
+ * This hook returns the (negated) value of the navigator.onLine property, and
+ * also monitors for changes.
+ */
+export const useIsOffline = () => {
+    const [offline, setOffline] = useState(
+        typeof window !== "undefined" && !window.navigator.onLine,
+    );
+
+    useEffect(() => {
+        const setUserOnline = () => setOffline(false);
+        const setUserOffline = () => setOffline(true);
+
+        window.addEventListener("online", setUserOnline);
+        window.addEventListener("offline", setUserOffline);
+
+        return () => {
+            window.removeEventListener("online", setUserOnline);
+            window.removeEventListener("offline", setUserOffline);
+        };
+    }, []);
+
+    return offline;
+};