From 11cd746784c3d14e9e2c2f36f5e3ae3f0ec82341 Mon Sep 17 00:00:00 2001
From: Guillermo Espinosa <juan.gmo1997@gmail.com>
Date: Thu, 5 Sep 2024 15:44:23 -0400
Subject: [PATCH] take out props from viewColumn

---
 packages/ui/package.json                |  2 +-
 packages/ui/src/DataTable/index.js      |  7 ++++++-
 packages/ui/src/DataTable/viewColumn.js | 14 +++++---------
 3 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/packages/ui/package.json b/packages/ui/package.json
index 06fe2c64..9b968759 100644
--- a/packages/ui/package.json
+++ b/packages/ui/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@teselagen/ui",
-  "version": "0.5.23-beta.7",
+  "version": "0.5.23-beta.8",
   "main": "./src/index.js",
   "type": "module",
   "exports": {
diff --git a/packages/ui/src/DataTable/index.js b/packages/ui/src/DataTable/index.js
index c3bb6ae0..f906cce4 100644
--- a/packages/ui/src/DataTable/index.js
+++ b/packages/ui/src/DataTable/index.js
@@ -538,7 +538,10 @@ const DataTable = ({
       schema.fields = [viewColumn, ...schema.fields];
     }
     if (isOpenable) {
-      schema.fields = [openColumn, ...schema.fields];
+      schema.fields = [
+        openColumn({ onDoubleClick, history }),
+        ...schema.fields
+      ];
     }
     // this must come before handling orderings.
     schema.fields = schema.fields.map(field => {
@@ -632,10 +635,12 @@ const DataTable = ({
     cellRenderer,
     convertedSchema,
     entities,
+    history,
     isInfinite,
     isOpenable,
     isSimple,
     isViewable,
+    onDoubleClick,
     showForcedHiddenColumns,
     tableConfig.columnOrderings,
     tableConfig.fieldOptions,
diff --git a/packages/ui/src/DataTable/viewColumn.js b/packages/ui/src/DataTable/viewColumn.js
index d5f88968..fb524135 100644
--- a/packages/ui/src/DataTable/viewColumn.js
+++ b/packages/ui/src/DataTable/viewColumn.js
@@ -11,20 +11,16 @@ export const viewColumn = {
     return <Icon className="dt-eyeIcon" icon="eye-open" />;
   }
 };
-export const openColumn = {
+export const openColumn = ({ onDoubleClick, history }) => ({
   ...viewColumn,
-  render: (val, record, rowInfo, props) => {
+  render: (val, record, rowInfo) => {
     return (
       <Tooltip content="Open">
         <Button
           onClick={e => {
             e.stopPropagation();
-            props.onDoubleClick &&
-              props.onDoubleClick(
-                rowInfo.original,
-                rowInfo.index,
-                props.history
-              );
+            onDoubleClick &&
+              onDoubleClick(rowInfo.original, rowInfo.index, history);
           }}
           minimal
           small
@@ -34,4 +30,4 @@ export const openColumn = {
       </Tooltip>
     );
   }
-};
+});