From 5e98ba54fa1e57bcd2fa8c3b8ffdd3757ece6fa4 Mon Sep 17 00:00:00 2001
From: Ritchie Martori <ritchie@voxel51.com>
Date: Fri, 2 Aug 2024 08:05:43 -0700
Subject: [PATCH 1/2] dashboard edit support

---
 .../SchemaIO/components/DashboardView.tsx     | 26 +++++++++++++++++++
 fiftyone/operators/types.py                   |  6 +++++
 2 files changed, 32 insertions(+)

diff --git a/app/packages/core/src/plugins/SchemaIO/components/DashboardView.tsx b/app/packages/core/src/plugins/SchemaIO/components/DashboardView.tsx
index 8d9e729268..4558de6512 100644
--- a/app/packages/core/src/plugins/SchemaIO/components/DashboardView.tsx
+++ b/app/packages/core/src/plugins/SchemaIO/components/DashboardView.tsx
@@ -19,6 +19,7 @@ import { Button } from ".";
 import { getPath, getProps } from "../utils";
 import { ObjectSchemaType, ViewPropsType } from "../utils/types";
 import DynamicIO from "./DynamicIO";
+import { Edit } from "@mui/icons-material";
 
 const AddItemCTA = ({ onAdd }) => {
   return (
@@ -69,6 +70,7 @@ export default function DashboardView(props: ViewPropsType) {
   const propertiesAsArray = [];
   const allow_addition = schema.view.allow_addition;
   const allow_deletion = schema.view.allow_deletion;
+  const allow_edit = schema.view.allow_edit;
 
   for (const property in properties) {
     propertiesAsArray.push({ id: property, ...properties[property] });
@@ -76,6 +78,17 @@ export default function DashboardView(props: ViewPropsType) {
   const panelId = usePanelId();
   const triggerPanelEvent = usePanelEvent();
 
+  const onEditItem = useCallback(
+    ({ id, path }) => {
+      if (schema.view.on_edit_item) {
+        triggerPanelEvent(panelId, {
+          operator: schema.view.on_edit_item,
+          params: { id, path },
+        });
+      }
+    },
+    [panelId, props, schema.view.on_edit_item, triggerPanelEvent]
+  );
   const onCloseItem = useCallback(
     ({ id, path }) => {
       if (schema.view.on_remove_item) {
@@ -185,6 +198,19 @@ export default function DashboardView(props: ViewPropsType) {
             >
               <DragHandle className="drag-handle">
                 <Typography>{property.title || id}</Typography>
+                {allow_edit && (
+                  <IconButton
+                    size="small"
+                    onMouseDown={(e) => e.stopPropagation()}
+                    onClick={(e) => {
+                      e.stopPropagation();
+                      onEditItem({ id, path: getPath(path, id) });
+                    }}
+                    sx={{ color: theme.text.secondary }}
+                  >
+                    <Edit />
+                  </IconButton>
+                )}
                 {allow_deletion && (
                   <IconButton
                     size="small"
diff --git a/fiftyone/operators/types.py b/fiftyone/operators/types.py
index ee4050ff79..bc02da0cfe 100644
--- a/fiftyone/operators/types.py
+++ b/fiftyone/operators/types.py
@@ -2099,18 +2099,24 @@ class DashboardView(View):
         on_layout_change (None): event triggered when the layout changes
         on_add_item (None): event triggered when an item is added
         on_remove_item (None): event triggered when an item is closed
+        on_edit_item (None): event triggered when an item is edited
+        allow_addition (True): whether to allow adding items
+        allow_deletion (True): whether to allow deleting items
+        allow_edit (True): whether to allow editing items
     """
 
     def __init__(self, **kwargs):
         super().__init__(**kwargs)
         self.allow_addition = kwargs.get("allow_addition", True)
         self.allow_deletion = kwargs.get("allow_deletion", True)
+        self.allow_edit = kwargs.get("allow_edit", True)
 
     def to_json(self):
         return {
             **super().to_json(),
             "allow_addition": self.allow_addition,
             "allow_deletion": self.allow_deletion,
+            "allow_edit": self.allow_edit,
         }
 
 

From 4b9d233cdaac96eb5ce5d0ec990f98df0d8a71e2 Mon Sep 17 00:00:00 2001
From: Ritchie Martori <ritchie@voxel51.com>
Date: Fri, 2 Aug 2024 08:15:53 -0700
Subject: [PATCH 2/2] dashboard edit fixes

---
 .../SchemaIO/components/DashboardView.tsx     | 54 ++++++++++---------
 1 file changed, 28 insertions(+), 26 deletions(-)

diff --git a/app/packages/core/src/plugins/SchemaIO/components/DashboardView.tsx b/app/packages/core/src/plugins/SchemaIO/components/DashboardView.tsx
index 4558de6512..50bd7b8e13 100644
--- a/app/packages/core/src/plugins/SchemaIO/components/DashboardView.tsx
+++ b/app/packages/core/src/plugins/SchemaIO/components/DashboardView.tsx
@@ -198,32 +198,34 @@ export default function DashboardView(props: ViewPropsType) {
             >
               <DragHandle className="drag-handle">
                 <Typography>{property.title || id}</Typography>
-                {allow_edit && (
-                  <IconButton
-                    size="small"
-                    onMouseDown={(e) => e.stopPropagation()}
-                    onClick={(e) => {
-                      e.stopPropagation();
-                      onEditItem({ id, path: getPath(path, id) });
-                    }}
-                    sx={{ color: theme.text.secondary }}
-                  >
-                    <Edit />
-                  </IconButton>
-                )}
-                {allow_deletion && (
-                  <IconButton
-                    size="small"
-                    onMouseDown={(e) => e.stopPropagation()}
-                    onClick={(e) => {
-                      e.stopPropagation();
-                      onCloseItem({ id, path: getPath(path, id) });
-                    }}
-                    sx={{ color: theme.text.secondary }}
-                  >
-                    <CloseIcon />
-                  </IconButton>
-                )}
+                <Box>
+                  {allow_edit && (
+                    <IconButton
+                      size="small"
+                      onMouseDown={(e) => e.stopPropagation()}
+                      onClick={(e) => {
+                        e.stopPropagation();
+                        onEditItem({ id, path: getPath(path, id) });
+                      }}
+                      sx={{ color: theme.text.secondary }}
+                    >
+                      <Edit />
+                    </IconButton>
+                  )}
+                  {allow_deletion && (
+                    <IconButton
+                      size="small"
+                      onMouseDown={(e) => e.stopPropagation()}
+                      onClick={(e) => {
+                        e.stopPropagation();
+                        onCloseItem({ id, path: getPath(path, id) });
+                      }}
+                      sx={{ color: theme.text.secondary }}
+                    >
+                      <CloseIcon />
+                    </IconButton>
+                  )}
+                </Box>
               </DragHandle>
               <Box sx={{ height: "calc(100% - 35px)", overflow: "auto" }}>
                 <DynamicIO