From f88cb30b44015b75ea21a26f647365e75ca16bf9 Mon Sep 17 00:00:00 2001
From: Jan Monschke <jan.monschke@elastic.co>
Date: Fri, 20 May 2022 10:45:15 +0200
Subject: [PATCH 1/2] fix: disable drag-ability and hover actions for agent
 statuses

The agent fields cannot be queried with ECS and therefore should not provide Filter In/Out functionality nor should users be able to add their representative fields to timeline investigations. Therefore users should not be able to add them to a timeline query by dragging them.
---
 .../body/renderers/agent_statuses.tsx         | 38 +++----------------
 1 file changed, 6 insertions(+), 32 deletions(-)

diff --git a/x-pack/plugins/security_solution/public/timelines/components/timeline/body/renderers/agent_statuses.tsx b/x-pack/plugins/security_solution/public/timelines/components/timeline/body/renderers/agent_statuses.tsx
index edc8faff1b5fc7..c459a9f05a6783 100644
--- a/x-pack/plugins/security_solution/public/timelines/components/timeline/body/renderers/agent_statuses.tsx
+++ b/x-pack/plugins/security_solution/public/timelines/components/timeline/body/renderers/agent_statuses.tsx
@@ -7,7 +7,6 @@
 
 import React from 'react';
 import { EuiFlexGroup, EuiFlexItem, EuiText } from '@elastic/eui';
-import { DefaultDraggable } from '../../../../../common/components/draggables';
 import { EndpointHostIsolationStatus } from '../../../../../common/components/endpoint/host_isolation';
 import { useHostIsolationStatus } from '../../../../../detections/containers/detection_engine/alerts/use_host_isolation_status';
 import { AgentStatus } from '../../../../../common/components/endpoint/agent_status';
@@ -33,26 +32,11 @@ export const AgentStatuses = React.memo(
   }) => {
     const { isIsolated, agentStatus, pendingIsolation, pendingUnisolation } =
       useHostIsolationStatus({ agentId: value });
-    const isolationFieldName = 'host.isolation';
     return (
       <EuiFlexGroup gutterSize="none">
         {agentStatus !== undefined ? (
           <EuiFlexItem grow={false}>
-            {isDraggable ? (
-              <DefaultDraggable
-                field={fieldName}
-                id={`event-details-value-default-draggable-${contextId}-${eventId}-${fieldName}-${value}`}
-                fieldType={fieldType}
-                isAggregatable={isAggregatable}
-                isDraggable={isDraggable}
-                tooltipContent={fieldName}
-                value={`${agentStatus}`}
-              >
-                <AgentStatus hostStatus={agentStatus} />
-              </DefaultDraggable>
-            ) : (
-              <AgentStatus hostStatus={agentStatus} />
-            )}
+            <AgentStatus hostStatus={agentStatus} />
           </EuiFlexItem>
         ) : (
           <EuiText>
@@ -60,21 +44,11 @@ export const AgentStatuses = React.memo(
           </EuiText>
         )}
         <EuiFlexItem grow={false}>
-          <DefaultDraggable
-            field={isolationFieldName}
-            id={`event-details-value-default-draggable-${contextId}-${eventId}-${isolationFieldName}-${value}`}
-            fieldType={fieldType}
-            isAggregatable={isAggregatable}
-            isDraggable={isDraggable}
-            tooltipContent={isolationFieldName}
-            value={`${isIsolated}`}
-          >
-            <EndpointHostIsolationStatus
-              isIsolated={isIsolated}
-              pendingIsolate={pendingIsolation}
-              pendingUnIsolate={pendingUnisolation}
-            />
-          </DefaultDraggable>
+          <EndpointHostIsolationStatus
+            isIsolated={isIsolated}
+            pendingIsolate={pendingIsolation}
+            pendingUnIsolate={pendingUnisolation}
+          />
         </EuiFlexItem>
       </EuiFlexGroup>
     );

From c33b0516b9aea644822cb426c5e82f91e8a472c8 Mon Sep 17 00:00:00 2001
From: Jan Monschke <jan.monschke@elastic.co>
Date: Fri, 20 May 2022 10:45:31 +0200
Subject: [PATCH 2/2] chore: make code more readable

---
 .../table/summary_value_cell.tsx              | 46 +++++++++++--------
 1 file changed, 26 insertions(+), 20 deletions(-)

diff --git a/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.tsx b/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.tsx
index 1c9c0292ed9124..d4677d22485b4b 100644
--- a/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.tsx
+++ b/x-pack/plugins/security_solution/public/common/components/event_details/table/summary_value_cell.tsx
@@ -16,6 +16,8 @@ import { AGENT_STATUS_FIELD_NAME } from '../../../../timelines/components/timeli
 
 const FIELDS_WITHOUT_ACTIONS: { [field: string]: boolean } = { [AGENT_STATUS_FIELD_NAME]: true };
 
+const style = { flexGrow: 0 };
+
 export const SummaryValueCell: React.FC<AlertSummaryRow['description']> = ({
   data,
   eventId,
@@ -25,32 +27,36 @@ export const SummaryValueCell: React.FC<AlertSummaryRow['description']> = ({
   timelineId,
   values,
   isReadOnly,
-}) => (
-  <>
-    <FieldValueCell
-      contextId={timelineId}
-      data={data}
-      eventId={eventId}
-      fieldFromBrowserField={fieldFromBrowserField}
-      linkValue={linkValue}
-      isDraggable={isDraggable}
-      style={{ flexGrow: 0 }}
-      values={values}
-    />
-    {timelineId !== TimelineId.active && !isReadOnly && !FIELDS_WITHOUT_ACTIONS[data.field] && (
-      <ActionCell
+}) => {
+  const hoverActionsEnabled = !FIELDS_WITHOUT_ACTIONS[data.field];
+
+  return (
+    <>
+      <FieldValueCell
         contextId={timelineId}
         data={data}
         eventId={eventId}
         fieldFromBrowserField={fieldFromBrowserField}
         linkValue={linkValue}
-        timelineId={timelineId}
+        isDraggable={isDraggable}
+        style={style}
         values={values}
-        applyWidthAndPadding={false}
-        hideAddToTimeline={false}
       />
-    )}
-  </>
-);
+      {timelineId !== TimelineId.active && !isReadOnly && hoverActionsEnabled && (
+        <ActionCell
+          contextId={timelineId}
+          data={data}
+          eventId={eventId}
+          fieldFromBrowserField={fieldFromBrowserField}
+          linkValue={linkValue}
+          timelineId={timelineId}
+          values={values}
+          applyWidthAndPadding={false}
+          hideAddToTimeline={false}
+        />
+      )}
+    </>
+  );
+};
 
 SummaryValueCell.displayName = 'SummaryValueCell';