From 56918bf5c29763761f151cce13d49d26519c1eb4 Mon Sep 17 00:00:00 2001
From: Damian Krysta <damian@krysta.dev>
Date: Mon, 31 Jan 2022 22:55:14 +0100
Subject: [PATCH] Add store in url for layout and all cameras

Signed-off-by: Damian Krysta <damian@krysta.dev>
---
 ui/src/App.tsx            | 30 ++++++++++++++++++------------
 ui/src/Live/Multiview.tsx | 36 +++++++++++++++++++-----------------
 ui/src/index.tsx          |  5 ++++-
 3 files changed, 41 insertions(+), 30 deletions(-)

diff --git a/ui/src/App.tsx b/ui/src/App.tsx
index 284bef5..0ed0cf1 100644
--- a/ui/src/App.tsx
+++ b/ui/src/App.tsx
@@ -11,7 +11,7 @@ import { useSnackbars } from "./snackbars";
 import { Camera, Session } from "./types";
 import ListActivity from "./List";
 import AppBar from "@mui/material/AppBar";
-import {BrowserRouter, Routes, Route, Link} from "react-router-dom";
+import {Routes, Route, Link, useSearchParams, useResolvedPath, useMatch} from "react-router-dom";
 import LiveActivity, { MultiviewChooser } from "./Live";
 import Drawer from "@mui/material/Drawer";
 import List from "@mui/material/List";
@@ -34,12 +34,16 @@ type Activity = "list" | "live";
 
 function App() {
   const [showMenu, toggleShowMenu] = useReducer((m: boolean) => !m, false);
+  const [searchParams, setSearchParams] = useSearchParams();
+
   const [showListSelectors, toggleShowListSelectors] = useReducer(
     (m: boolean) => !m,
     true
   );
-  const [activity, setActivity] = useState<Activity>("list");
-  const [multiviewLayoutIndex, setMultiviewLayoutIndex] = useState(0);
+  let resolved = useResolvedPath('live');
+  let match = useMatch({ path: resolved.pathname, end: true });
+  const [activity, setActivity] = useState<Activity>(match ? "live" : "list");
+  const [multiviewLayoutIndex, setMultiviewLayoutIndex] = useState(Number.parseInt(searchParams.get('layout') || '0', 10));
   const [session, setSession] = useState<Session | null>(null);
   const [cameras, setCameras] = useState<Camera[] | null>(null);
   const [timeZoneName, setTimeZoneName] = useState<string | null>(null);
@@ -145,14 +149,17 @@ function App() {
         activityMenu = (
           <MultiviewChooser
             layoutIndex={multiviewLayoutIndex}
-            onChoice={setMultiviewLayoutIndex}
+            onChoice={(value) => {
+              setMultiviewLayoutIndex(value)
+              setSearchParams({layout: value.toString()}, )
+            }}
           />
         );
         break;
     }
   }
   return (
-    <BrowserRouter>
+    <>
       <AppBar position="static">
         <MoonfireMenu
           loginState={loginState}
@@ -174,18 +181,17 @@ function App() {
         }}
       >
         <List>
-          <ListItem button key="list" onClick={() => clickActivity("list")}>
+          <ListItem button key="list" onClick={() => clickActivity("list")} component={Link} to="/">
             <ListItemIcon>
               <ListIcon />
             </ListItemIcon>
-            <Link to="/"><ListItemText primary="List view" /></Link>
+            <ListItemText primary="List view" />
           </ListItem>
-          <ListItem button key="live" onClick={() => clickActivity("live")}>
+          <ListItem button key="live" onClick={() => clickActivity("live")} component={Link} to="/live">
             <ListItemIcon>
               <Videocam />
             </ListItemIcon>
-
-            <Link to="/live"><ListItemText primary="Live view (experimental)" /></Link>
+            <ListItemText primary="Live view (experimental)" />
           </ListItem>
         </List>
       </Drawer>
@@ -211,10 +217,10 @@ function App() {
           </p>
         </Container>
       )}
-      <Routes >
+      <Routes>
         {fetchedCameras(cameras)}
       </Routes>
-    </BrowserRouter>
+    </>
   );
 }
 
diff --git a/ui/src/Live/Multiview.tsx b/ui/src/Live/Multiview.tsx
index 722bb36..54ead02 100644
--- a/ui/src/Live/Multiview.tsx
+++ b/ui/src/Live/Multiview.tsx
@@ -4,7 +4,7 @@
 
 import Select, { SelectChangeEvent } from "@mui/material/Select";
 import MenuItem from "@mui/material/MenuItem";
-import React, { useReducer } from "react";
+import React, {useReducer} from "react";
 import { Camera } from "../types";
 import { makeStyles } from "@mui/styles";
 import { Theme } from "@mui/material/styles";
@@ -91,13 +91,13 @@ export const MultiviewChooser = (props: MultiviewChooserProps) => {
     <Select
       id="layout"
       value={props.layoutIndex}
-      onChange={(e) =>
+      onChange={(e) => {
         props.onChoice(
           typeof e.target.value === "string"
             ? parseInt(e.target.value)
             : e.target.value
         )
-      }
+      }}
       size="small"
       sx={{
         // Hacky attempt to style for the app menu.
@@ -130,6 +130,7 @@ interface SelectOp {
   cameraIndex: number | null;
 }
 
+
 function selectedReducer(old: SelectedCameras, op: SelectOp): SelectedCameras {
   let selected = [...old]; // shallow clone.
   if (op.cameraIndex !== null) {
@@ -151,15 +152,17 @@ function selectedReducer(old: SelectedCameras, op: SelectOp): SelectedCameras {
  * as possible.
  */
 const Multiview = (props: MultiviewProps) => {
+  const [searchParams, setSearchParams] = useSearchParams();
 
   const [selected, updateSelected] = useReducer(
     selectedReducer,
-    Array(MAX_CAMERAS).fill(null)
+    searchParams.has('cams') ? JSON.parse(searchParams.get('cams') || "") : Array(MAX_CAMERAS).fill(null)
   );
-  const outerRef = React.useRef<HTMLDivElement>(null);
 
+  const outerRef = React.useRef<HTMLDivElement>(null);
   const classes = useStyles();
   const layout = LAYOUTS[props.layoutIndex];
+
   const monoviews = selected.slice(0, layout.cameras).map((e, i) => {
     // When a camera is selected, use the camera's index as the key.
     // This allows swapping cameras' positions without tearing down their
@@ -168,18 +171,24 @@ const Multiview = (props: MultiviewProps) => {
     // When no camera is selected, use the index within selected. (Actually,
     // -1 minus the index, to disambiguate between the two cases.)
     const key = e ?? -1 - i;
+
     return (
       <Monoview
         key={key}
         cameras={props.cameras}
         cameraIndex={e}
         renderCamera={props.renderCamera}
-        onSelect={(cameraIndex) =>
-          updateSelected({ selectedIndex: i, cameraIndex })
-        }
+        onSelect={(cameraIndex) => {
+          updateSelected({selectedIndex: i, cameraIndex})
+          searchParams.set('cams',  JSON.stringify(selectedReducer(selected, {selectedIndex: i, cameraIndex})))
+          setSearchParams(searchParams)
+        }}
       />
     );
   });
+
+
+
   return (
     <div className={classes.root} ref={outerRef}>
       <div className="mid">
@@ -200,24 +209,17 @@ interface MonoviewProps {
 
 /** A single pane of a Multiview, including its camera chooser. */
 const Monoview = (props: MonoviewProps) => {
-  const [searchParams, setSearchParams] = useSearchParams();
   const handleChange = (event: SelectChangeEvent<number | null>) => {
     const {
       target: { value },
     } = event;
-    if (value !== null && value !== undefined) {
-      setSearchParams(new URLSearchParams({cam: value.toString()}));
-    } else {
-      setSearchParams(new URLSearchParams({ }))
-    }
 
     props.onSelect(typeof value === "string" ? parseInt(value) : value);
   };
 
-  const fromQueryIndexOrNull = searchParams.has('cam') ? Number.parseInt(searchParams.get('cam') as string, 10) : null;
   const chooser = (
     <Select
-      value={props.cameraIndex == null ? fromQueryIndexOrNull: props.cameraIndex}
+      value={props.cameraIndex == null ? null: props.cameraIndex}
       onChange={handleChange}
       displayEmpty
       size="small"
@@ -238,7 +240,7 @@ const Monoview = (props: MonoviewProps) => {
     </Select>
   );
   return props.renderCamera(
-    props.cameraIndex === null ? fromQueryIndexOrNull === null ? null : props.cameras[fromQueryIndexOrNull] : props.cameras[props.cameraIndex],
+    props.cameraIndex === null ? null : props.cameras[props.cameraIndex],
     chooser
   );
 };
diff --git a/ui/src/index.tsx b/ui/src/index.tsx
index f9f4ad8..45b44b9 100644
--- a/ui/src/index.tsx
+++ b/ui/src/index.tsx
@@ -14,6 +14,7 @@ import ErrorBoundary from "./ErrorBoundary";
 import { SnackbarProvider } from "./snackbars";
 import AdapterDateFns from "@mui/lab/AdapterDateFns";
 import "./index.css";
+import { BrowserRouter } from "react-router-dom";
 
 const theme = createTheme({
   palette: {
@@ -34,7 +35,9 @@ ReactDOM.render(
         <ErrorBoundary>
           <LocalizationProvider dateAdapter={AdapterDateFns}>
             <SnackbarProvider autoHideDuration={5000}>
-              <App />
+              <BrowserRouter >
+                <App />
+              </BrowserRouter>
             </SnackbarProvider>
           </LocalizationProvider>
         </ErrorBoundary>