chore(weave): remove add row button from edit and confirm in add to d…

…ataset (#3783)
wandb · Feb 26, 2025 · 50c7cce · 50c7cce
1 parent 7a51fef
commit 50c7cce
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/weave-js/src/components/PagePanelComponents/Home/Browse3/datasets/DatasetPreview.tsx b/weave-js/src/components/PagePanelComponents/Home/Browse3/datasets/DatasetPreview.tsx
@@ -36,6 +36,7 @@ export const DatasetPreview: React.FC<DatasetPreviewProps> = ({
       datasetObject={datasetObject}
       isEditing={true}
       hideRemoveForAddedRows={true}
+      showAddRowButton={false}
     />
   );
 };
diff --git a/weave-js/src/components/PagePanelComponents/Home/Browse3/datasets/EditableDatasetView.tsx b/weave-js/src/components/PagePanelComponents/Home/Browse3/datasets/EditableDatasetView.tsx
@@ -59,6 +59,7 @@ export interface EditableDatasetViewProps {
   datasetObject: DatasetObjectVal;
   isEditing?: boolean;
   hideRemoveForAddedRows?: boolean;
+  showAddRowButton?: boolean;
 }
 
 interface OrderedRow {
@@ -70,6 +71,7 @@ export const EditableDatasetView: React.FC<EditableDatasetViewProps> = ({
   datasetObject,
   isEditing = false,
   hideRemoveForAddedRows = false,
+  showAddRowButton = true,
 }) => {
   const {useTableRowsQuery, useTableQueryStats} = useWFHooks();
   const [sortBy, setSortBy] = useState<SortBy[]>([]);
@@ -473,7 +475,7 @@ export const EditableDatasetView: React.FC<EditableDatasetViewProps> = ({
   const CustomFooter = useCallback(() => {
     return (
       <GridFooterContainer>
-        {isEditing && (
+        {isEditing && showAddRowButton && (
           <Box
             sx={{
               padding: '8px 16px',
@@ -499,7 +501,7 @@ export const EditableDatasetView: React.FC<EditableDatasetViewProps> = ({
         </Box>
       </GridFooterContainer>
     );
-  }, [isEditing, handleAddRowsClick]);
+  }, [isEditing, handleAddRowsClick, showAddRowButton]);
 
   return (
     <div style={{display: 'flex', flexDirection: 'column', height: '100%'}}>