Merge pull request 'extend toolbar styling, add support for atomic menu' (#539) from hsjobeki-main into main

pkgs/ui/src/app/layout.tsx

@@ -1,24 +1,17 @@
 "use client";
 import { Sidebar } from "@/components/sidebar";
 import { tw } from "@/utils/tailwind";
-import MenuIcon from "@mui/icons-material/Menu";
-import {
-  CssBaseline,
-  IconButton,
-  ThemeProvider,
-  useMediaQuery,
-} from "@mui/material";
+import { CssBaseline, ThemeProvider, useMediaQuery } from "@mui/material";
 import { StyledEngineProvider } from "@mui/material/styles";
 import axios from "axios";
 import localFont from "next/font/local";
-import Image from "next/image";
 import * as React from "react";
 import { Toaster } from "react-hot-toast";
 import "./globals.css";
 import { darkTheme, lightTheme } from "./theme/themes";
 
-import { WithAppState } from "@/components/hooks/useAppContext";
 import { ClanToolbar } from "@/components/clanToolbar";
+import { WithAppState } from "@/components/hooks/useAppContext";
 
 const roboto = localFont({
   src: [
@@ -70,29 +63,10 @@ export default function RootLayout({
                     !showSidebar && translate
                   } flex h-full w-full flex-col overflow-y-scroll transition-[margin] duration-150 ease-in-out`}
                 >
-                  <ClanToolbar />
-                  <div className="static top-0 mb-2 py-2">
-                    <div className="grid grid-cols-3">
-                      <div className="col-span-1">
-                        <IconButton
-                          hidden={showSidebar}
-                          onClick={() => setShowSidebar((c) => !c)}
-                        >
-                          {!showSidebar && <MenuIcon />}
-                        </IconButton>
-                      </div>
-                      <div className="col-span-1 block w-full bg-fixed text-center font-semibold dark:invert lg:hidden">
-                        <Image
-                          src="/favicon.png"
-                          alt="Clan Logo"
-                          width={58}
-                          height={58}
-                          priority
+                  <ClanToolbar
+                    isSidebarVisible={showSidebar}
+                    handleSidebar={setShowSidebar}
                   />
-                      </div>
-                    </div>
-                  </div>
-
                   <div className="px-1">
                     <div className="relative flex h-full flex-1 flex-col">
                       <main>{children}</main>

pkgs/ui/src/app/manage/create/page.tsx

@@ -0,0 +1,59 @@
+"use client";
+import { createFlake } from "@/api/flake/flake";
+import { HTTPValidationError } from "@/api/model";
+import { CreateClan, CreateFormValues } from "@/components/forms/createClan";
+import { clanErrorToast } from "@/error/errorToast";
+import { AxiosError } from "axios";
+import { FormProvider, useForm } from "react-hook-form";
+
+export default function Manage() {
+  const methods = useForm<CreateFormValues>({
+    defaultValues: {
+      flakeDir: "",
+      flakeTemplateUrl: "",
+    },
+  });
+  const { handleSubmit } = methods;
+  return (
+    <FormProvider {...methods}>
+      <form
+        onSubmit={handleSubmit(async (values) => {
+          console.log({ values });
+          try {
+            await createFlake(
+              {
+                url: values.flakeTemplateUrl,
+              },
+              {
+                flake_dir: values.flakeDir,
+              },
+            );
+          } catch (e) {
+            const error = e as AxiosError<HTTPValidationError>;
+            clanErrorToast(error);
+            const maybeDetail = error?.response?.data?.detail?.[0];
+
+            if (maybeDetail?.loc && maybeDetail?.msg) {
+              const urlError = error.response?.data.detail?.find((detail) =>
+                detail.loc.includes("url"),
+              );
+              urlError &&
+                methods.setError("flakeTemplateUrl", {
+                  message: urlError.msg,
+                });
+              const flakeDirError = error.response?.data.detail?.find(
+                (detail) => detail.loc.includes("flake_dir"),
+              );
+              flakeDirError &&
+                methods.setError("flakeDir", {
+                  message: flakeDirError.msg,
+                });
+            }
+          }
+        })}
+      >
+        <CreateClan methods={methods} />
+      </form>
+    </FormProvider>
+  );
+}

pkgs/ui/src/app/manage/join/page.tsx

@@ -1,3 +1,4 @@
+"use client";
 import JoinPrequel from "@/views/joinPrequel";
 
 export default function Page() {

pkgs/ui/src/app/manage/page.tsx

@@ -0,0 +1,22 @@
+import { Button } from "@mui/material";
+import Link from "next/link";
+
+export default function Manage() {
+  return (
+    <div>
+      Select
+      <Button>
+        <Link href="/manage/join">Join</Link>
+      </Button>
+      <Button>
+        <Link href="/manage/create">Create</Link>
+      </Button>
+      <ul>
+        <li>History</li>
+        <li>Recent History</li>
+        <li>Ancient History</li>
+        <li>Cosmic History</li>
+      </ul>
+    </div>
+  );
+}

pkgs/ui/src/components/clanToolbar/index.tsx

@@ -1,6 +1,7 @@
 import { useFlakeHistoryList } from "@/api/flake/flake";
 import DynamicFeedIcon from "@mui/icons-material/DynamicFeed";
-import { IconButton, LinearProgress } from "@mui/material";
+import MenuIcon from "@mui/icons-material/Menu";
+import { Button, Divider, LinearProgress } from "@mui/material";
 import Menu from "@mui/material/Menu";
 import MenuItem from "@mui/material/MenuItem";
 import * as React from "react";
@@ -9,57 +10,112 @@ interface ToolbarButtonProps {
   icon: React.ReactNode;
   onClick: (event: React.MouseEvent<HTMLButtonElement>) => void;
 }
-function ToolbarButton(props: ToolbarButtonProps) {
+export function ToolbarButton(props: ToolbarButtonProps) {
   const { icon, onClick } = props;
   return (
-    <div className="">
-      <IconButton onClick={onClick}>{icon}</IconButton>
-    </div>
+    <Button
+      sx={{
+        "& .MuiButton-startIcon": {
+          m: 0,
+        },
+      }}
+      onClick={onClick}
+      startIcon={icon}
+      variant="text"
+      color="inherit"
+      // fullWidth
+    />
   );
 }
+
+const ClanHistoryMenu = () => {
+  const { data, isLoading } = useFlakeHistoryList();
+
+  return (
+    <>
+      {isLoading ? (
+        <LinearProgress />
+      ) : (
+        data?.data.map((item, index) => <MenuItem key={index}>{item}</MenuItem>)
+      )}
+      {!isLoading && data?.data.length === 0 && (
+        <MenuItem>No Clan History</MenuItem>
+      )}
+    </>
+  );
+};
+
 type ToolbarItem = {
   icon: React.ReactNode;
+  menu: React.ReactNode;
 };
 const toolbarItems: ToolbarItem[] = [
   {
     icon: <DynamicFeedIcon />,
+    menu: <ClanHistoryMenu />,
   },
 ];
-export function ClanToolbar() {
-  const { data, isLoading } = useFlakeHistoryList();
+
+interface ClanToolbarProps {
+  isSidebarVisible: boolean;
+  handleSidebar: React.Dispatch<React.SetStateAction<boolean>>;
+}
+export function ClanToolbar(props: ClanToolbarProps) {
+  const { isSidebarVisible, handleSidebar } = props;
+
   const [anchorEl, setAnchorEl] = React.useState<null | HTMLElement>(null);
-  const open = Boolean(anchorEl);
-  const handleClick = (event: React.MouseEvent<HTMLButtonElement>) => {
+  const [openIdx, setOpenIdx] = React.useState<number | null>(null);
+
+  const handleClick = (
+    event: React.MouseEvent<HTMLButtonElement>,
+    idx: number,
+  ) => {
     setAnchorEl(event.currentTarget);
+    setOpenIdx(idx);
   };
   const handleClose = () => {
     setAnchorEl(null);
+    setOpenIdx(null);
   };
   return (
-    <div className="grid w-full auto-cols-min grid-flow-col grid-rows-1 place-items-end justify-end gap-2">
+    <div
+      className="flex border-x-0 border-b
+    border-t-0 border-solid border-neutral-80"
+    >
+      {!isSidebarVisible && (
+        <ToolbarButton
+          icon={<MenuIcon />}
+          onClick={() => handleSidebar((c) => !c)}
+        />
+      )}
+      <div
+        className="
+    grid w-full auto-cols-min grid-flow-col grid-rows-1 
+   justify-end gap-0
+    "
+      >
         {toolbarItems.map((item, index) => (
-        <ToolbarButton key={index} icon={item.icon} onClick={handleClick} />
-      ))}
+          <React.Fragment key={index}>
+            <Divider flexItem orientation="vertical" />
+            <ToolbarButton
+              icon={item.icon}
+              onClick={(ev) => handleClick(ev, index)}
+            />
+
             <Menu
               id="basic-menu"
               anchorEl={anchorEl}
-        open={open}
+              open={index == openIdx}
               onClose={handleClose}
               MenuListProps={{
                 "aria-labelledby": "basic-button",
               }}
             >
-        {isLoading ? (
-          <LinearProgress />
-        ) : (
-          data?.data.map((item, index) => (
-            <MenuItem key={index}>{item}</MenuItem>
-          ))
-        )}
-        {!isLoading && data?.data.length === 0 && (
-          <MenuItem>No Clan History</MenuItem>
-        )}
+              {item.menu}
             </Menu>
+          </React.Fragment>
+        ))}
+      </div>
     </div>
   );
 }

pkgs/ui/src/components/forms/createClan/index.tsx

@@ -0,0 +1,77 @@
+import CopyAllIcon from "@mui/icons-material/CopyAll";
+import SaveAltIcon from "@mui/icons-material/SaveAlt";
+import {
+  Button,
+  InputAdornment,
+  LinearProgress,
+  TextField,
+} from "@mui/material";
+import { Controller, UseFormReturn } from "react-hook-form";
+
+export type CreateFormValues = {
+  flakeTemplateUrl: string;
+  flakeDir: string;
+};
+
+interface CreateClanProps {
+  confirmAdornment?: React.ReactNode;
+  methods: UseFormReturn<CreateFormValues>;
+}
+
+export const CreateClan = (props: CreateClanProps) => {
+  const { methods } = props;
+  const {
+    control,
+    formState: { isSubmitting },
+  } = methods;
+  return (
+    <div>
+      <Controller
+        name="flakeTemplateUrl"
+        control={control}
+        render={({ field, fieldState }) => (
+          <TextField
+            id="flakeTemplateUrl"
+            error={Boolean(fieldState.error)}
+            label="Clan Template"
+            fullWidth
+            variant="standard"
+            InputProps={{
+              endAdornment: (
+                <InputAdornment position="end">
+                  <CopyAllIcon />
+                </InputAdornment>
+              ),
+            }}
+            helperText={fieldState.error?.message}
+            {...field}
+          />
+        )}
+      />
+      <Controller
+        name="flakeDir"
+        control={control}
+        render={({ field, fieldState }) => (
+          <TextField
+            id="flakeDir"
+            error={Boolean(fieldState.error)}
+            label="Directory"
+            fullWidth
+            variant="standard"
+            InputProps={{
+              endAdornment: (
+                <InputAdornment position="end">
+                  <SaveAltIcon />
+                </InputAdornment>
+              ),
+            }}
+            helperText={fieldState.error?.message}
+            {...field}
+          />
+        )}
+      />
+      <Button type="submit">Create</Button>
+      {isSubmitting && <LinearProgress />}
+    </div>
+  );
+};

pkgs/ui/src/components/sidebar/index.tsx

@@ -41,7 +41,7 @@ const menuEntries: MenuEntry[] = [
   {
     icon: <WorkspacesIcon />,
     label: "Manage",
-    to: "/join",
+    to: "/manage",
   },
   {
     icon: <BackupIcon />,

pkgs/ui/src/error/errorToast.ts

@@ -5,8 +5,10 @@ import { toast } from "react-hot-toast";
 export function clanErrorToast(error: AxiosError<HTTPValidationError>) {
   console.error({ error });
   const detail = error.response?.data.detail?.[0]?.msg;
+  const detailAlt = error.response?.data.detail as unknown as string;
   const cause = error.cause?.message;
   const axiosMessage = error.message;
-  const sanitizedMsg = detail || cause || axiosMessage || "Unexpected error";
+  const sanitizedMsg =
+    detail || detailAlt || cause || axiosMessage || "Unexpected error";
   toast.error(sanitizedMsg);
 }

pkgs/ui/src/views/createForm.tsx

@@ -1,56 +0,0 @@
-import { Input, InputAdornment, LinearProgress } from "@mui/material";
-import { Controller, useFormContext } from "react-hook-form";
-
-interface CreateFormProps {
-  confirmAdornment?: React.ReactNode;
-}
-
-export const CreateForm = (props: CreateFormProps) => {
-  const { confirmAdornment } = props;
-  const {
-    control,
-    formState: { isSubmitting },
-  } = useFormContext();
-  return (
-    <div>
-      <Controller
-        name="flakeUrl"
-        control={control}
-        render={({ field }) => (
-          <Input
-            disableUnderline
-            placeholder="url"
-            color="secondary"
-            aria-required="true"
-            {...field}
-            required
-            fullWidth
-            startAdornment={
-              <InputAdornment position="start">Clan</InputAdornment>
-            }
-          />
-        )}
-      />
-      <Controller
-        name="dest"
-        control={control}
-        render={({ field }) => (
-          <Input
-            sx={{ my: 2 }}
-            placeholder="Location"
-            color="secondary"
-            aria-required="true"
-            {...field}
-            required
-            fullWidth
-            startAdornment={
-              <InputAdornment position="start">Name</InputAdornment>
-            }
-            endAdornment={confirmAdornment}
-          />
-        )}
-      />
-      {isSubmitting && <LinearProgress />}
-    </div>
-  );
-};

pkgs/ui/src/views/joinForm.tsx

@@ -1,16 +1,16 @@
 import { Confirm } from "@/components/join/confirm";
+import PublicIcon from "@mui/icons-material/Public";
 import { Input, InputAdornment } from "@mui/material";
 import { Controller, useFormContext } from "react-hook-form";
 
 interface JoinFormProps {
-  confirmAdornment?: React.ReactNode;
   initialParams: {
     flakeUrl: string;
     flakeAttr: string;
   };
 }
 export const JoinForm = (props: JoinFormProps) => {
-  const { initialParams, confirmAdornment } = props;
+  const { initialParams } = props;
   const { control, formState, reset, getValues, watch } = useFormContext();
 
   return (
@@ -38,10 +38,11 @@ export const JoinForm = (props: JoinFormProps) => {
               {...field}
               required
               fullWidth
-              startAdornment={
-                <InputAdornment position="start">Clan</InputAdornment>
+              endAdornment={
+                <InputAdornment position="end">
+                  <PublicIcon />
+                </InputAdornment>
               }
-              endAdornment={confirmAdornment}
             />
           )}
         />

pkgs/ui/src/views/joinPrequel.tsx

@@ -1,29 +1,18 @@
 "use client";
-import {
-  IconButton,
-  InputAdornment,
-  MenuItem,
-  Select,
-  Typography,
-} from "@mui/material";
+import { Button, Typography } from "@mui/material";
 import { useSearchParams } from "next/navigation";
 import { Suspense, useState } from "react";
 
-import { createFlake } from "@/api/flake/flake";
 import { VmConfig } from "@/api/model";
 import { createVm } from "@/api/vm/vm";
-import { useAppState } from "@/components/hooks/useAppContext";
 import { Layout } from "@/components/join/layout";
 import { VmBuildLogs } from "@/components/join/vmBuildLogs";
-import { ChevronRight } from "@mui/icons-material";
 import { AxiosError } from "axios";
-import { Controller, FormProvider, useForm } from "react-hook-form";
+import { FormProvider, useForm } from "react-hook-form";
 import { toast } from "react-hot-toast";
-import { CreateForm } from "./createForm";
 import { JoinForm } from "./joinForm";
 
 export type FormValues = VmConfig & {
-  workflow: "join" | "create";
   flakeUrl: string;
   dest?: string;
 };
@@ -34,48 +23,21 @@ export default function JoinPrequel() {
   const flakeAttr = queryParams.get("attr") || "default";
   const initialParams = { flakeUrl, flakeAttr };
 
-  const { setAppState } = useAppState();
-
   const methods = useForm<FormValues>({
     defaultValues: {
       flakeUrl: "",
       dest: undefined,
-      workflow: "join",
       cores: 4,
       graphics: true,
       memory_size: 2048,
     },
   });
 
-  const { control, watch, handleSubmit } = methods;
+  const { handleSubmit } = methods;
 
   const [vmUuid, setVmUuid] = useState<string | null>(null);
   const [showLogs, setShowLogs] = useState<boolean>(false);
 
-  const workflow = watch("workflow");
-
-  const WorkflowAdornment = (
-    <InputAdornment position="end">
-      <Controller
-        name="workflow"
-        control={control}
-        render={({ field }) => (
-          <Select
-            {...field}
-            label="workflow"
-            variant="standard"
-            disableUnderline
-          >
-            <MenuItem value={"join"}>Join</MenuItem>
-            <MenuItem value={"create"}>Create</MenuItem>
-          </Select>
-        )}
-      />
-      <IconButton type={"submit"}>
-        <ChevronRight />
-      </IconButton>
-    </InputAdornment>
-  );
   return (
     <Layout
       header={
@@ -84,11 +46,8 @@ export default function JoinPrequel() {
           className="w-full text-center"
           sx={{ textTransform: "capitalize" }}
         >
-          {workflow}{" "}
-          <Typography variant="h4" className="font-bold" component={"span"}>
           Clan.lol
         </Typography>
-        </Typography>
       }
     >
       <Suspense fallback="Loading">
@@ -98,24 +57,6 @@ export default function JoinPrequel() {
           <FormProvider {...methods}>
             <form
               onSubmit={handleSubmit(async (values) => {
-                if (workflow === "create") {
-                  try {
-                    await createFlake(
-                      {
-                        url: values.flakeUrl,
-                      },
-                      {
-                        flake_dir: values.dest || "myclan",
-                      },
-                    );
-                    setAppState((s) => ({ ...s, isJoined: true }));
-                  } catch (error) {
-                    toast.error(
-                      `Error: ${(error as AxiosError).message || ""}`,
-                    );
-                  }
-                }
-                if (workflow === "join") {
                 console.log("JOINING");
                 console.log(values);
                 try {
@@ -135,23 +76,13 @@ export default function JoinPrequel() {
                     toast.error("Could not join");
                   }
                 } catch (error) {
-                    toast.error(
-                      `Error: ${(error as AxiosError).message || ""}`,
-                    );
-                  }
+                  toast.error(`Error: ${(error as AxiosError).message || ""}`);
                 }
               })}
               className="w-full max-w-2xl justify-self-center"
             >
-              {workflow == "join" && (
-                <JoinForm
-                  initialParams={initialParams}
-                  confirmAdornment={WorkflowAdornment}
-                />
-              )}
-              {workflow == "create" && (
-                <CreateForm confirmAdornment={WorkflowAdornment} />
-              )}
+              <JoinForm initialParams={initialParams} />
+              <Button type="submit">Join</Button>
             </form>
           </FormProvider>
         )}