From 5d01389f3b1696be2f7e98a84bdf845ff32263d2 Mon Sep 17 00:00:00 2001
From: Ivan Li <ivanli2048@gmail.com>
Date: Fri, 1 Oct 2021 15:40:21 +0800
Subject: [PATCH] =?UTF-8?q?feat(pipelines):=20=E5=88=97=E8=A1=A8=E6=B7=BB?=
 =?UTF-8?q?=E5=8A=A0=E5=8F=B3=E9=94=AE=E8=8F=9C=E5=8D=95=E3=80=82?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/commons/auth/login.tsx      |  1 -
 src/pipelines/pipeline-list.tsx | 71 +++++++++++++++++++--------------
 2 files changed, 42 insertions(+), 30 deletions(-)

diff --git a/src/commons/auth/login.tsx b/src/commons/auth/login.tsx
index 0159a99..3bbac9c 100644
--- a/src/commons/auth/login.tsx
+++ b/src/commons/auth/login.tsx
@@ -2,7 +2,6 @@ import makeStyles from "@mui/styles/makeStyles";
 import { FC, Fragment, useEffect, useRef } from "react";
 import { useAuth } from "./auth.provider";
 const useStyles = makeStyles((theme) => {
-  debugger;
   return {
     iframe: {
       height: "300px",
diff --git a/src/pipelines/pipeline-list.tsx b/src/pipelines/pipeline-list.tsx
index f555e4f..a2a983a 100644
--- a/src/pipelines/pipeline-list.tsx
+++ b/src/pipelines/pipeline-list.tsx
@@ -10,11 +10,13 @@ import {
   Menu,
   MenuItem,
   PopoverPosition,
+  styled,
 } from "@mui/material";
-import { FC, MouseEventHandler, useMemo, useState } from "react";
+import { FC, MouseEventHandler, useMemo, useState, MouseEvent } from "react";
 import { Pipeline, Project } from "../generated/graphql";
 import { CallMerge, Edit } from "@mui/icons-material";
 import { any, values } from "ramda";
+import { Divider } from "@material-ui/core";
 
 interface Props {
   projectId: string;
@@ -47,17 +49,37 @@ export const PipelineList: FC<Props> = ({ projectId }) => {
     }));
   }, [data]);
 
-  const [contextMenu, setContextMenu] = useState<PopoverPosition>();
+  const [contextMenu, setContextMenu] = useState<[PopoverPosition, Pipeline]>();
+  const { navigate, url } = useRouter();
 
-  const handleContextMenu: MouseEventHandler = (event) => {
+  const handleContextMenu = (event: MouseEvent, pipeline: Pipeline | false) => {
     event.preventDefault();
-    setContextMenu({ top: event.clientY - 4, left: event.clientX - 2 });
+    if (pipeline) {
+      setContextMenu([
+        { top: event.clientY - 4, left: event.clientX - 2 },
+        pipeline,
+      ]);
+    } else {
+      setContextMenu(undefined);
+    }
   };
 
   const handleClose = () => {
     setContextMenu(undefined);
   };
 
+  const modify = () => {
+    navigate({
+      url: url({
+        name: "edit-pipeline",
+        params: {
+          pipelineId: contextMenu![1].id,
+          projectId: contextMenu![1].project.id,
+        },
+      }),
+    });
+  };
+
   return (
     <>
       <List>
@@ -66,7 +88,7 @@ export const PipelineList: FC<Props> = ({ projectId }) => {
             name="pipeline-commits"
             params={{ pipelineId: pipeline.id, projectId: projectId }}
             key={pipeline.id}
-            onContextMenu={handleContextMenu}
+            onContextMenu={(ev) => handleContextMenu(ev, pipeline)}
           >
             <Item pipeline={pipeline} />
           </Link>
@@ -76,33 +98,29 @@ export const PipelineList: FC<Props> = ({ projectId }) => {
         keepMounted
         open={!!contextMenu}
         onClose={handleClose}
+        onContextMenu={(ev) => handleContextMenu(ev, false)}
         anchorReference="anchorPosition"
-        anchorPosition={
-          contextMenu && any(Boolean, values(contextMenu))
-            ? contextMenu
-            : undefined
-        }
+        anchorPosition={contextMenu?.[0]}
       >
-        <MenuItem onClick={handleClose}>Copy</MenuItem>
-        <MenuItem onClick={handleClose}>Print</MenuItem>
-        <MenuItem onClick={handleClose}>Highlight</MenuItem>
-        <MenuItem onClick={handleClose}>Email</MenuItem>
+        <MenuItem onClick={handleClose}>Runtime Config</MenuItem>
+        <MenuItem
+          onClick={() => {
+            modify();
+            handleClose();
+          }}
+        >
+          Pipeline Config
+        </MenuItem>
+        <Divider />
+        <MenuItem sx={{ color: "error.main" }} onClick={handleClose}>
+          Delete
+        </MenuItem>
       </Menu>
     </>
   );
 };
 
 const Item = ({ pipeline }: { pipeline: Pipeline }) => {
-  const { navigate, url } = useRouter();
-  const modify: MouseEventHandler = (ev) => {
-    ev.preventDefault();
-    navigate({
-      url: url({
-        name: "edit-pipeline",
-        params: { pipelineId: pipeline.id, projectId: pipeline.project.id },
-      }),
-    });
-  };
   return (
     <ListItem button>
       <ListItemText
@@ -114,11 +132,6 @@ const Item = ({ pipeline }: { pipeline: Pipeline }) => {
           </Typography>
         }
       />
-      <ListItemSecondaryAction>
-        <IconButton edge="end" aria-label="edit" onClick={modify} size="large">
-          <Edit />
-        </IconButton>
-      </ListItemSecondaryAction>
     </ListItem>
   );
 };