From d1b93eacaca7cf01df23da1abdf5fc445d240e6d Mon Sep 17 00:00:00 2001
From: SrGooglo <srgooglo@ragestudio.net>
Date: Sat, 24 Jun 2023 11:58:31 +0000
Subject: [PATCH] rewrite application navigation handlers

---
 .../settings/components/sessions/index.jsx    |  2 +-
 .../FeaturedEventAnnouncement/index.jsx       |  2 +-
 packages/app/src/components/Footer/index.jsx  |  2 +-
 .../src/components/Layout/sidebar/index.jsx   |  4 +-
 .../app/src/components/Navigation/index.jsx   |  2 +-
 .../components/Player/MediaPlayer/index.jsx   |  2 +-
 .../PlaylistTimelineEntry/index.jsx           |  2 +-
 .../app/src/components/PostCard/index.jsx     |  2 +-
 .../app/src/components/RenderError/index.jsx  |  2 +-
 .../src/components/WidgetsWrapper/index.jsx   |  2 +-
 packages/app/src/pages/[param].jsx            |  2 +-
 .../src/pages/home/components/feed/index.jsx  |  2 +-
 packages/app/src/pages/lyrics/index.jsx       |  2 +-
 .../components/dashboard/releases/index.jsx   |  2 +-
 .../src/pages/music/components/feed/index.jsx |  2 +-
 .../components/TracksUploads/index.jsx        |  2 +-
 packages/app/src/pages/tv/tabs/feed/index.jsx |  2 +-
 packages/app/src/router.jsx                   | 88 ++++++++++---------
 packages/app/src/theme/fixments.less          | 22 +++++
 19 files changed, 85 insertions(+), 61 deletions(-)

diff --git a/packages/app/constants/settings/components/sessions/index.jsx b/packages/app/constants/settings/components/sessions/index.jsx
index b187f994..54e20b4c 100755
--- a/packages/app/constants/settings/components/sessions/index.jsx
+++ b/packages/app/constants/settings/components/sessions/index.jsx
@@ -4,7 +4,7 @@ import { Button } from "antd"
 export default (props) => {
     return <Button
         onClick={() => {
-            app.setLocation("/security/sessions")
+            app.location.push("/security/sessions")
             props.ctx.close()
         }}
     >
diff --git a/packages/app/src/components/FeaturedEventAnnouncement/index.jsx b/packages/app/src/components/FeaturedEventAnnouncement/index.jsx
index 4ae1fc60..ea7cdf58 100755
--- a/packages/app/src/components/FeaturedEventAnnouncement/index.jsx
+++ b/packages/app/src/components/FeaturedEventAnnouncement/index.jsx
@@ -12,7 +12,7 @@ export default (props) => {
             return false
         }
 
-        app.setLocation(`/featured-event/${props.data?._id}`)
+        app.location.push(`/featured-event/${props.data?._id}`)
     }
 
     return <div
diff --git a/packages/app/src/components/Footer/index.jsx b/packages/app/src/components/Footer/index.jsx
index 359cebff..557e20fe 100755
--- a/packages/app/src/components/Footer/index.jsx
+++ b/packages/app/src/components/Footer/index.jsx
@@ -16,7 +16,7 @@ export default (props) => {
 
             if (link.location) {
                 linkProps.onClick = () => {
-                    app.setLocation(link.location)
+                    app.location.push(link.location)
                 }
             }
 
diff --git a/packages/app/src/components/Layout/sidebar/index.jsx b/packages/app/src/components/Layout/sidebar/index.jsx
index f377c83b..6623cdda 100755
--- a/packages/app/src/components/Layout/sidebar/index.jsx
+++ b/packages/app/src/components/Layout/sidebar/index.jsx
@@ -311,11 +311,11 @@ export default class Sidebar extends React.Component {
 
 		if (typeof this.state.pathResolvers === "object") {
 			if (typeof this.state.pathResolvers[e.key] !== "undefined") {
-				return window.app.setLocation(`/${this.state.pathResolvers[e.key]}`, 150)
+				return window.app.location.push(`/${this.state.pathResolvers[e.key]}`, 150)
 			}
 		}
 
-		return window.app.setLocation(`/${e.key}`, 150)
+		return window.app.location.push(`/${e.key}`, 150)
 	}
 
 	toggleExpanded = (to, force) => {
diff --git a/packages/app/src/components/Navigation/index.jsx b/packages/app/src/components/Navigation/index.jsx
index 1582188b..e9307bac 100755
--- a/packages/app/src/components/Navigation/index.jsx
+++ b/packages/app/src/components/Navigation/index.jsx
@@ -8,7 +8,7 @@ import "./index.less"
 
 export default class NavigationMenu extends React.Component {
     onClick = (id) => {
-        window.app.setLocation(`/${id}`)
+        window.app.location.push(`/${id}`)
         this.props.close()
     }
 
diff --git a/packages/app/src/components/Player/MediaPlayer/index.jsx b/packages/app/src/components/Player/MediaPlayer/index.jsx
index 3ef2ea7d..a8a4ebc4 100755
--- a/packages/app/src/components/Player/MediaPlayer/index.jsx
+++ b/packages/app/src/components/Player/MediaPlayer/index.jsx
@@ -46,7 +46,7 @@ export class AudioPlayer extends React.Component {
     }
 
     openVisualizer = () => {
-        app.setLocation("/lyrics")
+        app.location.push("/lyrics")
     }
 
     inviteSync = () => {
diff --git a/packages/app/src/components/PlaylistTimelineEntry/index.jsx b/packages/app/src/components/PlaylistTimelineEntry/index.jsx
index 24cf9256..310595ae 100644
--- a/packages/app/src/components/PlaylistTimelineEntry/index.jsx
+++ b/packages/app/src/components/PlaylistTimelineEntry/index.jsx
@@ -13,7 +13,7 @@ export default (props) => {
     }
 
     const navigateToPlaylist = () => {
-        app.setLocation(`/play/${data._id}`)
+        app.location.push(`/play/${data._id}`)
     }
 
     return <div className="playlistTimelineEntry">
diff --git a/packages/app/src/components/PostCard/index.jsx b/packages/app/src/components/PostCard/index.jsx
index e4b6d8c1..b634bf8e 100755
--- a/packages/app/src/components/PostCard/index.jsx
+++ b/packages/app/src/components/PostCard/index.jsx
@@ -36,7 +36,7 @@ const messageRegexs = [
     {
         regex: /(@[a-zA-Z0-9_]+)/gi,
         fn: (key, result) => {
-            return <a key={key} onClick={() => window.app.setLocation(`/@${result[1].substr(1)}`)}>{result[1]}</a>
+            return <a key={key} onClick={() => window.app.location.push(`/@${result[1].substr(1)}`)}>{result[1]}</a>
         }
     },
 ]
diff --git a/packages/app/src/components/RenderError/index.jsx b/packages/app/src/components/RenderError/index.jsx
index 567668a1..56f63c29 100755
--- a/packages/app/src/components/RenderError/index.jsx
+++ b/packages/app/src/components/RenderError/index.jsx
@@ -33,7 +33,7 @@ export default (props) => {
 	}
 
 	const onClickGoMain = () => {
-		window.app.setLocation(config.app.mainPath ?? "/main")
+		window.app.location.push(config.app.mainPath ?? "/main")
 	}
 	const onClickReload = () => {
 		window.location.reload()
diff --git a/packages/app/src/components/WidgetsWrapper/index.jsx b/packages/app/src/components/WidgetsWrapper/index.jsx
index 5bbd6e7f..876e3d8f 100644
--- a/packages/app/src/components/WidgetsWrapper/index.jsx
+++ b/packages/app/src/components/WidgetsWrapper/index.jsx
@@ -232,7 +232,7 @@ export default class WidgetsWrapper extends React.Component {
                         id: "settings",
                         icon: "Settings",
                         onClick: () => {
-                            app.setLocation("/settings?tab=widgets")
+                            app.location.push("/settings?tab=widgets")
                         }
                     },
                 ]}
diff --git a/packages/app/src/pages/[param].jsx b/packages/app/src/pages/[param].jsx
index 4648611f..1a9b4c94 100755
--- a/packages/app/src/pages/[param].jsx
+++ b/packages/app/src/pages/[param].jsx
@@ -7,7 +7,7 @@ export default (props) => {
     if (entryParam.startsWith("@")) {
         const username = entryParam.replace("@", "")
 
-        window.app.setLocation(`/account/${username}`, {
+        window.app.location.push(`/account/${username}`, {
             state: {
                 "noTransition": true,
             }
diff --git a/packages/app/src/pages/home/components/feed/index.jsx b/packages/app/src/pages/home/components/feed/index.jsx
index ff2ec41c..19b7c9a0 100755
--- a/packages/app/src/pages/home/components/feed/index.jsx
+++ b/packages/app/src/pages/home/components/feed/index.jsx
@@ -13,7 +13,7 @@ const emptyListRender = () => {
         </h2>
 
         <p>
-            Search for new people to follow on <a onClick={() => app.setLocation("/?type=global")}>global</a> tab, and start view their posts.
+            Search for new people to follow on <a onClick={() => app.location.push("/?type=global")}>global</a> tab, and start view their posts.
         </p>
     </div>
 }
diff --git a/packages/app/src/pages/lyrics/index.jsx b/packages/app/src/pages/lyrics/index.jsx
index b8cf7da3..de699cf1 100644
--- a/packages/app/src/pages/lyrics/index.jsx
+++ b/packages/app/src/pages/lyrics/index.jsx
@@ -620,7 +620,7 @@ export default class SyncLyrics extends React.Component {
         // listen when user exit full screen to exit cinematic mode
         document.addEventListener("fullscreenchange", () => {
             if (!document.fullscreenElement) {
-                app.backLocation()
+                app.location.back()
             }
         })
 
diff --git a/packages/app/src/pages/music/components/dashboard/releases/index.jsx b/packages/app/src/pages/music/components/dashboard/releases/index.jsx
index 72bb5b7e..32c19509 100644
--- a/packages/app/src/pages/music/components/dashboard/releases/index.jsx
+++ b/packages/app/src/pages/music/components/dashboard/releases/index.jsx
@@ -88,7 +88,7 @@ const openPlaylistCreator = ({
 }
 
 const navigateToPlaylist = (playlist_id) => {
-    return app.setLocation(`/play/${playlist_id}`)
+    return app.location.push(`/play/${playlist_id}`)
 }
 
 export default (props) => {
diff --git a/packages/app/src/pages/music/components/feed/index.jsx b/packages/app/src/pages/music/components/feed/index.jsx
index 4fbc060a..cf41c507 100755
--- a/packages/app/src/pages/music/components/feed/index.jsx
+++ b/packages/app/src/pages/music/components/feed/index.jsx
@@ -143,7 +143,7 @@ const PlaylistItem = (props) => {
             return props.onClick(playlist)
         }
 
-        return app.setLocation(`/play/${playlist._id}`)
+        return app.location.push(`/play/${playlist._id}`)
     }
 
     const onClickPlay = (e) => {
diff --git a/packages/app/src/pages/music/creator/components/TracksUploads/index.jsx b/packages/app/src/pages/music/creator/components/TracksUploads/index.jsx
index 8251a08d..5f9ce038 100644
--- a/packages/app/src/pages/music/creator/components/TracksUploads/index.jsx
+++ b/packages/app/src/pages/music/creator/components/TracksUploads/index.jsx
@@ -332,7 +332,7 @@ export default (props) => {
 
     return <div className="tracksUploads">
         <p>
-            Uploading files that are not permitted by our <a onClick={() => app.setLocation("/terms")}>Terms of Service</a> may result in your account being suspended.
+            Uploading files that are not permitted by our <a onClick={() => app.location.push("/terms")}>Terms of Service</a> may result in your account being suspended.
         </p>
 
         <div className="uploadBox">
diff --git a/packages/app/src/pages/tv/tabs/feed/index.jsx b/packages/app/src/pages/tv/tabs/feed/index.jsx
index 62a11d3e..1d9b7d06 100755
--- a/packages/app/src/pages/tv/tabs/feed/index.jsx
+++ b/packages/app/src/pages/tv/tabs/feed/index.jsx
@@ -88,7 +88,7 @@ export default (props) => {
     const [L_Streams, R_Streams, E_Streams] = app.cores.api.useRequest(Livestream.getLivestreams)
 
     const onClickItem = (livestream) => {
-        app.setLocation(`/live/${livestream.streamUrl}`)
+        app.location.push(`/live/${livestream.streamUrl}`)
     }
 
     if (E_Streams) {
diff --git a/packages/app/src/router.jsx b/packages/app/src/router.jsx
index 2cdac8a0..cdfd795c 100755
--- a/packages/app/src/router.jsx
+++ b/packages/app/src/router.jsx
@@ -73,8 +73,8 @@ function generatePageElementWrapper(route, element, bindProps) {
         if (routeDeclaration) {
             if (!bindProps.user && (window.location.pathname !== config.app?.authPath)) {
                 if (!routeDeclaration.public) {
-                    if (typeof window.app.setLocation === "function") {
-                        window.app.setLocation(config.app?.authPath ?? "/login")
+                    if (typeof window.app.location.push === "function") {
+                        window.app.location.push(config.app?.authPath ?? "/login")
                         return <div />
                     }
 
@@ -92,7 +92,7 @@ function generatePageElementWrapper(route, element, bindProps) {
                         status="403"
                         title="403"
                         subTitle="Sorry, you are not authorized to access this page."
-                        extra={<Button type="primary" onClick={() => window.app.setLocation("/")}>Back Home</Button>}
+                        extra={<Button type="primary" onClick={() => window.app.location.push("/")}>Back Home</Button>}
                     />
                 }
             }
@@ -140,52 +140,54 @@ function generatePageElementWrapper(route, element, bindProps) {
 
 export const PageRender = React.memo((props) => {
     const navigate = useNavigate()
-    app.location = useLocation()
 
-    React.useEffect(() => {
-        app.setLocation = async (to, state = {}) => {
-            // clean double slashes
-            to = to.replace(/\/{2,}/g, "/")
+    async function setLocation(to, state = {}) {
+        // clean double slashes
+        to = to.replace(/\/{2,}/g, "/")
 
-            // if state is a number, it's a delay
-            if (typeof state !== "object") {
-                state = {}
-            }
-
-            const transitionDuration = app.cores.style.getValue("page-transition-duration") ?? "250ms"
-
-            state.transitionDelay = Number(transitionDuration.replace("ms", ""))
-
-            app.eventBus.emit("router.navigate", to, {
-                state,
-            })
-
-            app.location.lastPathname = app.location.pathname
-
-            if (state.transitionDelay >= 100) {
-                await new Promise((resolve) => {
-                    setTimeout(() => {
-                        resolve()
-                    }, state.transitionDelay)
-                })
-            }
-
-            return navigate(to, {
-                state
-            })
+        // if state is a number, it's a delay
+        if (typeof state !== "object") {
+            state = {}
         }
 
-        app.backLocation = (state = {}) => {
-            let to = app.location.lastPathname
+        const transitionDuration = app.cores.style.getValue("page-transition-duration") ?? "250ms"
 
-            if (!to || to === app.location.pathname) {
-                console.warn("No last location found, redirecting to /")
-                to = "/"
-            }
+        state.transitionDelay = Number(transitionDuration.replace("ms", ""))
 
-            return navigate(to, {
-                state
-            })
+        app.eventBus.emit("router.navigate", to, {
+            state,
+        })
+
+        app.location.last = window.location
+
+        if (state.transitionDelay >= 100) {
+            await new Promise((resolve) => setTimeout(resolve, state.transitionDelay))
+        }
+
+        return navigate(to, {
+            state
+        })
+    }
+
+    async function backLocation() {
+        const transitionDuration = app.cores.style.getValue("page-transition-duration") ?? "250ms"
+
+        app.eventBus.emit("router.navigate")
+
+        app.location.last = window.location
+
+        if (transitionDuration >= 100) {
+            await new Promise((resolve) => setTimeout(resolve, transitionDuration))
+        }
+
+        return window.history.back()
+    }
+
+    React.useEffect(() => {
+        app.location = {
+            last: window.location,
+            push: setLocation,
+            back: backLocation,
         }
     }, [])
 
diff --git a/packages/app/src/theme/fixments.less b/packages/app/src/theme/fixments.less
index 50a6dc0c..3bc35c20 100755
--- a/packages/app/src/theme/fixments.less
+++ b/packages/app/src/theme/fixments.less
@@ -529,4 +529,26 @@
             }
         }
     }
+}
+
+.adm-action-sheet {
+    .adm-action-sheet-button-item-wrapper {
+        border-bottom-color: var(--border-color);
+
+        .adm-action-sheet-button-item {
+            .adm-action-sheet-button-item-name {
+                color: var(--text-color);
+
+                span {
+                    color: currentColor;
+                }
+            }
+        }
+
+        .adm-action-sheet-button-item-danger {
+            .adm-action-sheet-button-item-name {
+                color: var(--adm-color-danger);
+            }
+        }
+    }
 }
\ No newline at end of file